pyflakes icon indicating copy to clipboard operation
pyflakes copied to clipboard

Run pyflakes against all code in the virtualenv

Open jayvdb opened this issue 9 years ago • 7 comments
trafficstars

jayvdb avatar May 07 '16 02:05 jayvdb

Here is what it looks like after #61 is merged: https://travis-ci.org/jayvdb/pyflakes/builds/128447040

jayvdb avatar May 07 '16 02:05 jayvdb

Note that complete log is nicely 'folded'/hidden on Python 3.5+ as it has less flakes and thus fits within the allowed size for the log in the Travis web UI.

jayvdb avatar May 07 '16 02:05 jayvdb

just rebased.

jayvdb avatar Jul 05 '16 15:07 jayvdb

Can you explain why this is necessary? I feel like this is going to add a lot of noise in the travis logs.

sigmavirus24 avatar Jul 05 '16 16:07 sigmavirus24

The answer to the "why" question should probably go in the commit log message for posterity

bitglue avatar Jul 05 '16 21:07 bitglue

It would have blocked both of the regressions that occurred in 1.2 .

jayvdb avatar Jul 05 '16 23:07 jayvdb

@bitglue is correct. The why for this should be in the commit log and the file should have a comment telling people this is meant to prevent regressions that will only be caught on Travis (not locally). That comment will ideally cause people to look at the commit log and see your reasoning for this.

sigmavirus24 avatar Jul 06 '16 18:07 sigmavirus24

I'm planning to make an actual framework for tracking differences which would obsolete this

asottile avatar Nov 27 '22 21:11 asottile