Travis CI: Add flake8 jobs for Python 2.7 and Python 3.7#21953
Travis CI: Add flake8 jobs for Python 2.7 and Python 3.7#21953cclauss wants to merge 2 commits intonodejs:masterfrom
Conversation
|
My gut feeling says that I’d prefer #21952 (but it’s nice to have two options laid out!). /cc @nodejs/build-files @nodejs/python |
|
I tend to like this one because the committer gets a "fast fail". |
| - make -j2 V= | ||
| script: | ||
| - make -j2 test-ci | ||
| - os: linux |
There was a problem hiding this comment.
Should be covered by the lint-ci job
| - flake8 --version | ||
| - EXCLUDE=./deps/npm/node_modules/node-gyp,./deps/v8,./src/noperfctr_macros.py,./src/notrace_macros.py,./tools | ||
| - flake8 . --count --exclude=${EXCLUDE} --select=E901,E999,F821,F822,F823 --show-source --statistics | ||
| - os: linux |
There was a problem hiding this comment.
Not needed ATM. There are many script that are incompatible anyway.
| - export CXX="ccache clang++ -Qunused-arguments" | ||
| - export CC="ccache clang -Qunused-arguments -Wno-unknown-warning-option" | ||
| - export JOBS=2 | ||
| - export JOBS=4 |
There was a problem hiding this comment.
Should check why this was 2 before changing
|
@cclauss thanks again. |
refack
added
build
|
Is there a way to notify the committer that their PR contains errors detected by https://ci.nodejs.org? My sense is that my PR could contain errors but Travis CI and GitHub would be reporting all green. |
|
@cclauss The Node.js CI jobs should be posting status back to the associated PR. |
4 participants
An alternative approach to #21952 that runs flake8 in parallel Travis CI jobs on both Python 2.7 and Python 3.7. The Python 3.7 job is run in allow_failures mode.
See #21952 for more details on the --exclude and --select choices. A mix of these two PRs is also possible.
Checklist
make -j4 test(UNIX), orvcbuild test(Windows) passes