Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

Run flake8 tests to find Python syntax errors and undefined names #1217

Merged
merged 7 commits into from
Jul 10, 2019
Merged

Run flake8 tests to find Python syntax errors and undefined names #1217

Changes from 6 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
94fd107
Run flake8 tests to find Python syntax errors and undefined names
cclauss Jun 26, 2019
63ab6fe
python3 -m pip install flake8
cclauss Jun 26, 2019
fdbe52a
—user
cclauss Jun 27, 2019
f17ef31
python3 -m flake8
cclauss Jun 27, 2019
5d5ea22
IGNORE=./tools/nni_annotation/*:F821,./examples/trials/mnist-nas/mnis…
cclauss Jul 2, 2019
792c9a9
IGNORE=./tools/nni_annotation/testcase:F821,./examples/trials/mnist-n…
cclauss Jul 2, 2019
be83bd7
flake8 . --per-file-ignores=$IGNORE
cclauss Jul 2, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions azure-pipelines.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,11 @@ jobs:
- script: |
source install.sh
displayName: 'Install nni toolkit via source code'
- script: |
python3 -m pip install flake8 --user
IGNORE=./tools/nni_annotation/testcase:F821,./examples/trials/mnist-nas/mnist.py:F821
Copy link
Contributor

@chicm-ms chicm-ms Jul 2, 2019 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just checked locally, should change to --per-file-ignores

--per-file-ignores ./tools/nni_annotation/*:F821,./examples/trials/mnist-nas/mnist.py:F821

python3 -m flake8 . --count --ignore=$IGNORE --select=E9,F63,F72,F82 --show-source --statistics
displayName: 'Run flake8 tests to find Python syntax errors and undefined names'
- script: |
cd test
source unittest.sh
Expand Down