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
Fix parsing of multi-options for pycodestyle #112
Conversation
18664cf
to
e1efbe9
Compare
e1efbe9
to
e308003
Compare
The test failures are due to missing pythons: |
0870b74
to
eea05f4
Compare
60410fa
to
eea05f4
Compare
15a3d09
to
5bfe748
Compare
5bfe748
to
b2f5be5
Compare
You can't rely on tox alone to test the various versions of python because they might not be installed. Instead specify a build matrix by hand. This also allows for a cleaner after_script for coveralls.
b2f5be5
to
50b3562
Compare
This is blocked by #114 to make the tests green. Merging using rebase removes that commit, otherwise this needs to be rebased. |
Do we really need to check against python 3.3? Is there any OS in which python 3.3 is the default version? |
I'm not sure, but I didn't want to change that since it's currently in the testing matrix. But that should be in another PR, right? |
Ping @diraol how do we get this merged? |
This correctly parses options of the form "E302,W405", which before this was split into individual characters. This resulted in all errors and/or being considered when ignoring/selecting. Since specifying just "E" disables all errors, having a list of characters meant that it always encountered an individual "E" and thus disabled all errors.
50b3562
to
4c19d4c
Compare
@maxnordlund It is on the hands of @klen as far as I know. |
Any update? |
@maxnordlund Thank you for the PR. |
Np |
This correctly parses options of the form "E302,W405", which before this was split into individual characters. This resulted in all errors and/or being considered when ignoring/selecting.
Since specifying just "E" disables all errors, having a list of characters meant that it always encountered an individual "E" and thus disabled all errors.