Skip to content

Commit 65da01b

Browse files
authored
[CI] Fix commit range for clang-format check (#1185)
It turns out that my original implementation was correct and I just mis-understand the double dot commit range description from ProGit https://git-scm.com/book/en/v2/Git-Tools-Revision-Selection. Signed-off-by: Alexey Bader <[email protected]>
1 parent f2214a0 commit 65da01b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/workflows/clang-format.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ jobs:
2121

2222
- name: Run clang-format for the patch
2323
run: |
24-
git diff -U0 --no-color origin/${{github.base_ref}}..HEAD | ./clang/tools/clang-format/clang-format-diff.py -p1 -binary clang-format-9 > ./clang-format.patch
24+
git diff -U0 --no-color origin/${{github.base_ref}}...HEAD | ./clang/tools/clang-format/clang-format-diff.py -p1 -binary clang-format-9 > ./clang-format.patch
2525
2626
# Add patch with formatting fixes to CI job artifacts
2727
- uses: actions/upload-artifact@v1

0 commit comments

Comments
 (0)