未验证 提交 9d0d4a16 编写于 作者: N Nyakku Shigure 提交者: GitHub

[CodeStyle] fix cpplint hook not working on CI (#46136)

* update cpplint hook script

* update pre-commit config

* try trigger cpplint on ci

* Revert "try trigger cpplint on ci"

This reverts commit 895853453c5f68b587914713e313c92abb69fc45.

* empty commit, test=document_fix
上级 12c17448
...@@ -51,6 +51,10 @@ repos: ...@@ -51,6 +51,10 @@ repos:
entry: bash ./tools/codestyle/cpplint_pre_commit.hook entry: bash ./tools/codestyle/cpplint_pre_commit.hook
language: system language: system
files: \.(c|cc|cxx|cpp|cu|h|hpp|hxx)$ files: \.(c|cc|cxx|cpp|cu|h|hpp|hxx)$
args:
- --extensions=c,cc,cxx,cpp,cu,cuh,h,hpp,hxx,kps
- --filter=-readability/fn_size,-build/include_what_you_use,-build/c++11,-whitespace/parens
- --quiet
- repo: local - repo: local
hooks: hooks:
- id: pylint-doc-string - id: pylint-doc-string
......
#!/bin/bash #!/bin/bash
TOTAL_ERRORS=0
readonly VERSION="1.6.0" readonly VERSION="1.6.0"
version=$(cpplint --version) version=$(cpplint --version)
if [[ ! $TRAVIS_BRANCH ]]; then if ! [[ $version == *"$VERSION"* ]]; then
# install cpplint on local machine.
if ! [[ $version == *"$VERSION"* ]]; then
pip install cpplint==1.6.0 pip install cpplint==1.6.0
fi
# diff files on local machine.
files=$(git diff --cached --name-status | awk '$1 != "D" {print $2}')
else
# diff files between PR and latest commit on Travis CI.
branch_ref=$(git rev-parse "$TRAVIS_BRANCH")
head_ref=$(git rev-parse HEAD)
files=$(git diff --name-status $branch_ref $head_ref | awk '$1 != "D" {print $2}')
fi fi
# The trick to remove deleted files: https://stackoverflow.com/a/2413151
for file in $files; do
if [[ $file =~ ^(patches/.*) ]]; then
continue;
else
cpplint --filter=-readability/fn_size,-build/include_what_you_use,-build/c++11,-whitespace/parens $file;
TOTAL_ERRORS=$(expr $TOTAL_ERRORS + $?);
fi
done
exit $TOTAL_ERRORS cpplint $@
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册