Skip to content
This repository has been archived by the owner on Apr 22, 2020. It is now read-only.

Commit

Permalink
sanity-check.sh: disable some spurious warnings
Browse files Browse the repository at this point in the history 
* The default behaviour of pydocstyle is to ignore "test_" files when
  given a directory to check. Follow its example and ignore "test_"
  files in the invocation we use too.

* Ignore pycodestyle "E203" warnings - sometimes they are spurious
  (PyCQA/pycodestyle#373).
  • Loading branch information
@jonathanhaigh-arm
jonathanhaigh-arm committed Jan 29, 2019
1 parent 9ecf51a commit 3d7d23b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions ci/sanity-check/sanity-check.sh
View file
Original file line number Diff line number Diff line change
Expand Up @@ -119,10 +119,10 @@ printf "%s" "$PYTHON_FILES" | xargs --no-run-if-empty black --line-length 79 -v

# Run pycodestyle on python files
printf "Running pycodestyle on python files...\n"
printf "%s" "$PYTHON_FILES" | xargs --no-run-if-empty pycodestyle || rc=1
printf "%s" "$PYTHON_FILES" | xargs --no-run-if-empty pycodestyle --ignore=E203 || rc=1

# Run pydocstyle on python files
printf "Running pydocstyle on python files...\n"
printf "%s" "$PYTHON_FILES" | xargs --no-run-if-empty pydocstyle || rc=1
printf "%s" "$PYTHON_FILES" | xargs --no-run-if-empty pydocstyle --match='(?!.*test_).*\.py' || rc=1

exit "$rc"

0 comments on commit 3d7d23b

Please sign in to comment.