search

plone / plone.recipe.codeanalysis

provides static code analysis for Buildout-based Python projects, including flake8, JSHint, CSS Lint, and other code checks
https://pypi.org/project/plone.recipe.codeanalysis/
11 stars 8 forks source link

Using flake8-select=E123 breaks reporing violations alltogether. #212

Open tisto opened 6 years ago

tisto commented 6 years ago

When using

flake8-select = E123

in plone.recipe.codeanalysis (in plone.restapi, see https://github.com/plone/plone.restapi/pull/460/files), no violations are reported.

gforcada commented 6 years ago

Could it be that there are,way too many errors?

I've seen flake8 choke when the output is too big...

Just guessing.

tisto commented 6 years ago

@gforcada you mean too many E123 errors? Could be, will check...The other violations were just 3.

tisto commented 6 years ago

@gforcada was worth a shot but that's not the problem. Running:

bin/flake8 --select E123 src/

Does not return anything. I fixed all those violations myself when I added the check.