Skip to content
Permalink
Browse files

travis.yml: explicitly ignore flake8 rules

When using the '--ignore' flag to flake8 on the command-line, ignore
rules specified in the .flake8 config are ignored. We therefore need to
explicitly list W503 and W504 again when invoking flake8 on
files/folders for which docstrings don't pass the test yet.

This commit also re-enables flake8 to run on all source files again, not
just those which were modified from master.

Signed-off-by: Niklas Koep <niklas.koep@gmail.com>
  • Loading branch information
nkoep committed Feb 14, 2020
1 parent a0a4025 commit 5b701d151411c9f032bf83e5c9a78a544a60846e
Showing with 2 additions and 4 deletions.
  1. +1 −1 .flake8
  2. +1 −3 .travis.yml
@@ -2,5 +2,5 @@
application_import_names = geomstats
docstring-convention = numpy
exclude = examples/*.ipynb
ignore = W503,W504
ignore = W503, W504
import_order_style = smarkets
@@ -15,9 +15,7 @@ install:
pip install -q -r $req
done
script:
- >
flake8 $(git diff --name-only master HEAD geomstats examples tests)
--ignore=D
- flake8 --ignore=D,W503,W504 geomstats examples tests
- flake8 geomstats/geometry
- |
if [[ $TRAVIS_PYTHON_VERSION != 3.8 ||

0 comments on commit 5b701d1

Please sign in to comment.
You can’t perform that action at this time.