Any idea why `flake8` reports could be off by one

Question

curved-television-6568 · Answer

Locally I get ```src python pants engine rules test py 1012 9 F811 redefinition of unused dup a from line 1008```

curved-television-6568 · Answer

In CI however I get ```src python pants engine rules test py 1011 9 F811 redefinition of unused dup a from line 1007```

curved-television-6568 · Answer

I ll just cover both lines then but I find it weird

bitter-ability-32190 · Answer

Are you saying `flake8` is flaky

bitter-ability-32190 · Answer

At the risk of asking a very lame question do you have everything committed and pushed

curved-television-6568 · Answer

yup pretty sure unless there s some files hidden due to gitignore

bitter-ability-32190 · Answer

Got the PR

curved-television-6568 · Answer

also hoping I don t have any config that gets picked up in ~ but I don t see how that would affect the line numbering

bitter-ability-32190 · Answer

Yeah I wouldn t imagine it does

curved-television-6568 · Answer

curved-television-6568 · Answer

Latest CI runs don t show that issue now as I ve added the lint ignore comment on both lines

curved-television-6568 · Answer

I have one hypothesis about line endings which I haven t looked at yet if there s a single line using only ` r` or something funky like that if that would even work

bitter-ability-32190 · Answer

Doesn t answer your question but IMO F811 can be silenced if `mypy` is also warning No reason to have 2 linters for the same thing and I expect `mypy` is more powerful when it comes to detection

bitter-ability-32190 · Answer

I d approve that PR triumph

curved-television-6568 · Answer

good point

curved-television-6568 · Answer

yeah they both complained about the dupe stuck out tongue

bitter-ability-32190 · Answer

I ve been trying to move us off `pylint` and move all the checks to either `mypy` or `flake8`

bitter-ability-32190 · Answer

in company repo

bitter-ability-32190 · Answer

Ohh hmm one difference might be Python version The AST module has improved over time and `flake8` might be using different attrs in later python

curved-television-6568 · Answer

AAahhh right That sounds plausible that it would report the line with the decorator or the actual line for the method def differently

hundreds-father-404 · Answer

That is probably it CI uses 3 7