Permalink
Browse files

Move the Travis flake8 logic into test/lint.sh.

Remove Python 3.
  • Loading branch information...
Andy Chu
Andy Chu committed Feb 2, 2018
1 parent e91eb28 commit 95e2e329151f560e434fd0b475810828553ae33d
Showing with 27 additions and 13 deletions.
  1. +1 −13 .travis.yml
  2. +26 −0 test/lint.sh
View
@@ -2,23 +2,11 @@ language: python
cache: pip
python:
- 2.7
- 3.6
#- nightly
#- pypy
#- pypy3
matrix:
allow_failures:
- python: 3.6
install:
#- pip install -r requirements.txt
- pip install flake8 # pytest # add another testing frameworks later
before_script:
# stop the build if there are Python syntax errors or undefined names
- flake8 . --count --exclude=./Python-2.7.13 --select=E901,E999,F821,F822,F823 --show-source --statistics
# exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
- flake8 . --count --exclude=./Python-2.7.13 --exit-zero --max-complexity=10 --max-line-length=127 --statistics
script:
- test/lint.sh
- test/lint.sh travis
notifications:
on_success: change
on_failure: change # `always` will be the setting once code changes slow down
View
@@ -87,4 +87,30 @@ find-tabs() {
xargs grep -n $'\t'
}
bin-flake8() {
local ubuntu_flake8=~/.local/bin/flake8
if test -f "$ubuntu_flake8"; then
$ubuntu_flake8 "$@"
else
# Assume it's in $PATH, like on Travis.
flake8 "$@"
fi
}
flake8-all() {
local -a dirs=(asdl bin core osh)
# stop the build if there are Python syntax errors or undefined names
bin-flake8 "${dirs[@]}" \
--count --select=E901,E999,F821,F822,F823 --show-source --statistics
# exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
bin-flake8 "${dirs[@]}" \
--count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
}
# Hook for travis
travis() {
flake8-all
}
"$@"

0 comments on commit 95e2e32

Please sign in to comment.