-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
putty is not flake8 v3 compatible #14
Comments
After fixing all the other problems that have emerged since the last build, I assume that this is because pywikibot's tox.ini is installing flake8 v3. Need to test that theory yet. |
flake8 documentation about the change: http://flake8.pycqa.org/en/latest/plugin-development/cross-compatibility.html#option-handling-on-flake8-2-and-3 Related fixes in pep8-naming: |
Also, let me point out the log above includes |
Yea, the fix for pywikibot was to remove the It was easy to bypass the flake8 v3 option handling problem in the opening bug report, but as flake8 v3 is not using pep8 for the StyleGuide, file parsing and other structural classes, getting a hook inside of flake8 will need a lot of work. However I am still debating in #15 whether this flake8 is needed in a flake8 v3 world. |
For now I have updated setup.py in master to indicate it is incompatible with v3. |
Apparently 19bc89a is not enough to prevent flake8-putty being used with flake8-v3, as seen at https://phabricator.wikimedia.org/T141257 (the workaround is to pin |
And flake8 v3 is not compatible with Python 2.6
|
Would love to see this extension working with flake8 v3, I need it to filter specific "imported but not used" errors (based on the imported module) in my project. I would use v2 but that renders my own custom checker plugin useless. |
Hi @omriharel , what is your own custom checker plugin, and why it is useless on v2? See my response at #15 (comment) Another critical bug just reported: https://gitlab.com/pycqa/flake8/issues/214 And many more by @asottile about the unusable state of the legacy API, without a replacement API. flake8-putty fiddles inside the stack frames of flake8 and pycodestyle, so it is not sensible making it work when flake8 v3 internals are very likely to change. But if you really, really need it, I can at least investigate how much of -putty can be achieved without fiddling inside flake8/pycodestyle. |
Hi @jayvdb, I think it may be useless on v2 due to my own fault: I wasn't able to understand the plugin documentation very well (and assumed v3 was stable for some reason), so I just delved into its code and tried to figure out how pyflakes and pep8 themselves are plugged into flake8. My plugin is essentially a collection of checkers added on top of the default plugins (each in their own top level function being registered as a setuptools entry point, similar to the above examples), looking for team and project-specific infractions (and common gotchas) in our medium-sized codebase. I wasn't able to figure out how to properly make it work with v2, and between this and my particular filtering need, I'd rather just disable the unused import checks until I'm able to use flake8-putty. This isn't something I really, really need, but would enjoy seeing if and when it becomes possible - thank you for the kind offer, however :) |
Hi, @jayvdb do you have any plans to continue working on this? |
See #15 (comment) The v2 compatible version of putty is a hack, fiddling with the stack to bypass some of flake8 in order for this plugin to work. This is reasonable to maintain because v2 is very stable, and it is unlikely that upstream will change its internals, so the hack wont break. If I provide v3 support using a hack, it is quite likely that it will break, and I am then committed to try and find a solution. Also, flake8 v3 was a large rewrite which removed the section of code that I was using to get access to flake8 internals, so it will be much harder to reconstruct the hack on flake8 v3, and I'd prefer to not build a hacky version for flake8 v3 anyway. I hope that flake8 will eventually allow plugins to achieve the same result using a stable API. |
Also, fwiw, the other very important flake8 plugin for me is |
I published a similar flake8 extension, which is compatible with flake8 3, it's flake8-per-file-ignores. |
Great @snoack , I would be happy to accept a README update here linking to your project. |
Cool. There you go: #21 |
flake8_putty is broken for pywikibot, likely due to a change in either flake8 or pep8/pycodestyle.
investigating...
The text was updated successfully, but these errors were encountered: