Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: tox.ini

Issue 29569561: Noissue - Replace flake8-putty with flake8-per-file-ignores, update to flake8 3 (buildtools) (Closed)
Patch Set: Created Oct. 8, 2017, 5:50 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ensure_dependencies.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tox.ini
===================================================================
--- a/tox.ini
+++ b/tox.ini
@@ -3,15 +3,15 @@
skipsdist = true
[flake8]
-putty-ignore =
+per-file-ignores =
build.py : A102,A104,A107,A201,A206,A302,E501,E711,F401,N802,N803,N806
chainedconfigparser.py : A107,E501
ensure_dependencies.py : A102,A107,A108,A302,E129,E501,E713,E721,F821
localeTools.py : A103,A104,A107,A206,A301,A302,E129,E501,E711,E713,F401,N802,N803,N806
packager.py : A102,A107,A206,A302,E501,E711,N802,N803,N806
- packagerChrome.py : A101,A103,A104,A107,A108,A111,A112,A302,E501,E711,F841,N802,N803,N806
- packagerGecko.py : A101,A104,A105,A107,A108,A201,A302,E501,E711,E713,F401,N802,N803,N806
- packagerSafari.py : A101,A107,A301,A302,E501,N802,N803,N806
+ packagerChrome.py : A101,A104,A107,A108,A111,A112,A302,E501,E711,F841,N802,N803,N806
+ packagerGecko.py : A101,A104,A107,A108,A201,A302,E501,E711,E713,F401,N802,N803,N806
+ packagerSafari.py : A107,A302,E501,N802,N803,N806
publicSuffixListUpdater.py : A108,E501,F821,N802,N803
releaseAutomation.py : A102,A107,A108,A302,E501,F401,N803,N806
@@ -21,9 +21,8 @@
deps =
pytest
jinja2
- # flake8-putty is not yet compatible with flake8 3.0
- flake8==2.*
- flake8-putty
+ flake8
+ flake8-per-file-ignores
pep8-naming
hg+https://hg.adblockplus.org/codingtools#egg=flake8-eyeo&subdirectory=flake8-eyeo
commands =
« no previous file with comments | « ensure_dependencies.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld