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

2017-12-20 Thread Raphael Hertzog
Hello, On Tue, 19 Dec 2017, Ville Skyttä wrote: > I see a bunch of errors related to use of bare except there. Full list Yes, me too. Those appeared recently. > Subject: [PATCH] Exclude .tox and docs/conf.py from flake8 > > --- > tox.ini | 2 +- Merged your patch. On Tue, 19 Dec 2017, Paul

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,

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

2017-12-18 Thread Ville Skyttä
--- tox.ini | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tox.ini b/tox.ini index b8da0bd..04e5e39 100644 --- a/tox.ini +++ b/tox.ini @@ -57,7 +57,7 @@ deps = [flake8] max-complexity = 12 max-line-length = 80 -exclude =