Skip to content

Commit

Permalink
Manually ignore clang-format things
Browse files Browse the repository at this point in the history
  • Loading branch information
GeorgeLyon committed Oct 18, 2023
1 parent a076bbb commit f3b22b0
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
1 change: 0 additions & 1 deletion .devcontainer/default/VSCode/_clang-format

This file was deleted.

5 changes: 4 additions & 1 deletion .github/workflows/buildAndTest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@ jobs:
# Base ref is the target branch, in text form (not hash)
PR_BASE: ${{ github.base_ref }}
run: |
# Run clang-format
if [ -z "$PR_BASE" ]; then
DIFF_COMMIT_NAME="HEAD^"
else
Expand Down Expand Up @@ -73,6 +72,10 @@ jobs:
run: |
# Run clang-format
git clang-format $DIFF_COMMIT
# clang-format unnecessarily formats some JSON in this directory, and disagrees with VSCode's formatting rules
git checkout HEAD -- .devcontainer
git diff --ignore-submodules > clang-format.patch
if [ -s clang-format.patch ]; then
echo "Clang-format found formatting problems in the following " \
Expand Down

0 comments on commit f3b22b0

Please sign in to comment.