Re: [PATCH] Exclude .tox, distro_tracker/vendor, and docs/conf.py from flake8

2017-12-19 Thread Ville Skyttä
On Tue, Dec 19, 2017 at 11:13 AM, Raphael Hertzog wrote: > On Mon, 18 Dec 2017, Ville Skyttä wrote: >> -exclude = >> .git,.ropeproject,__pycache__,distro_tracker/project/settings/local.py,*/migrations/*.py >> +exclude = >>

Re: [PATCH] Exclude .tox, distro_tracker/vendor, and docs/conf.py from flake8

2017-12-19 Thread Paul Wise
On Tue, Dec 19, 2017 at 5:13 PM, Raphael Hertzog wrote: > But why distro_tracker/vendor? That sounds like embedded code copies so it should be checked/fixed by the upstream projects rather than by distro-tracker. > And why is it needed at all? I was not getting any warning from those >

Re: [PATCH] Exclude .tox, distro_tracker/vendor, and docs/conf.py from flake8

2017-12-19 Thread Raphael Hertzog
On Mon, 18 Dec 2017, Ville Skyttä wrote: > -exclude = > .git,.ropeproject,__pycache__,distro_tracker/project/settings/local.py,*/migrations/*.py > +exclude = > .git,.ropeproject,.tox,__pycache__,distro_tracker/project/settings/local.py,distro_tracker/vendor,docs/conf.py,*/migrations/*.py Ok,