Skip to content

Commit

Permalink
Automationstatic (#232)
Browse files Browse the repository at this point in the history
* use flake8 config to apply exclusions, remove travis wait

* pin pylint 1.9.2 for CI
  • Loading branch information
williexu authored Jul 16, 2018
1 parent 5a2c3eb commit 6b489fc
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 4 deletions.
13 changes: 13 additions & 0 deletions .flake8
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,16 @@ ignore =
F401, # imported but unused, too many violations, to be removed in the future
F811, # redefinition of unused, to be removed in the future
C901 # code flow is too complex, too many violations, to be removed in the future
exclude =
*/vendored_sdks
docs
scripts
./src/eventgrid/azext_eventgrid/mgmt/eventgrid
./src/botservice/azext_bot/botservice
./src/dns/azext_dns/dns
./src/managementgroups/azext_managementgroups/managementgroups
./src/managementpartner/azext_managementpartner/managementpartner
./src/rdbms_vnet/azext_rdbms_vnet/postgresql
./src/rdbms_vnet/azext_rdbms_vnet/mysql
./src/signalr/azext_signalr/signalr
./src/subscription/azext_subscription/subscription
6 changes: 3 additions & 3 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,16 +4,16 @@ language: python
services:
- docker
install:
- pip install pylint flake8 requests wheel==0.30.0 -q
- pip install pylint==1.9.2 flake8 requests wheel==0.30.0 -q
jobs:
include:
- stage: verify
env: PURPOSE='SourceStatic'
script: travis_wait ./scripts/ci/test_static.sh
script: ./scripts/ci/test_static.sh
python: 2.7
- stage: verify
env: PURPOSE='SourceStatic'
script: travis_wait ./scripts/ci/test_static.sh
script: ./scripts/ci/test_static.sh
python: 3.6
- stage: verify
env: PURPOSE='SourceTests'
Expand Down
2 changes: 1 addition & 1 deletion scripts/ci/test_static.sh
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ pylint ./src/*/azext_*/ --ignore=$PYLINT_EXCLUDES,vendored_sdks --ignore-pattern

echo "Pylint OK."
echo "Running flake8 on extensions..."
flake8 --statistics --exclude=$FLAKE8_EXCLUDES,vendored_sdks --append-config=./.flake8 ./src/*/azext_*/
flake8 --statistics --append-config=./.flake8 ./src/*/azext_*/
echo "Flake8 OK."

# Run pylint/flake8 on CI files
Expand Down

0 comments on commit 6b489fc

Please sign in to comment.