mirror of
https://github.com/QuasarApp/qca.git
synced 2025-04-26 03:24:31 +00:00
Add .git-blame-ignore-revs and some instructions for clang-format
This commit is contained in:
parent
dfb96ac596
commit
0684db8255
2
.git-blame-ignore-revs
Normal file
2
.git-blame-ignore-revs
Normal file
@ -0,0 +1,2 @@
|
||||
# _clang_format added
|
||||
f62a8ee8f7e81a1b573c335ded3326d8ee985ab4
|
14
README.clang-format
Normal file
14
README.clang-format
Normal file
@ -0,0 +1,14 @@
|
||||
We introduced clang-format mandatory usage in September 2020.
|
||||
If you want git blame to ignore the revision in which we did the mass change you can do
|
||||
git config blame.ignoreRevsFile .git-blame-ignore-revs
|
||||
on your clone
|
||||
|
||||
To get the clang-format warnings locally instead at CI time we recommend you
|
||||
to copy the hooks/pre-commit to your .git
|
||||
cp hooks/pre-commit .git/hooks/
|
||||
|
||||
We are using clang-format 10 on CI. Unfortunately clang-format is not totally
|
||||
compatible with older versions of itself. If CI gives you trouble but your local
|
||||
clang-format disagrees, just apply the changes suggested by CI and then commit
|
||||
with the --no-verify flag. If you get stuck, don't hesitate to ask the reviewer
|
||||
to help and they will reformat your commits :)
|
10
hooks/pre-commit
Normal file
10
hooks/pre-commit
Normal file
@ -0,0 +1,10 @@
|
||||
#!/usr/bin/env bash
|
||||
|
||||
readonly output=$(git clang-format -v --diff)
|
||||
|
||||
if [[ "$output" == *"no modified files to format"* ]]; then exit 0; fi
|
||||
if [[ "$output" == *"clang-format did not modify any files"* ]]; then exit 0; fi
|
||||
|
||||
echo "ERROR: you need to run git clang-format on your commit"
|
||||
echo " git clang-format -f is potentially what you want"
|
||||
exit 1
|
Loading…
x
Reference in New Issue
Block a user