Skip to content

Conversation

@choucavalier
Copy link

No description provided.

mtmiller and others added 30 commits June 2, 2016 14:32
Check for prohibited identifiers occuring to the lhs of an assignment
operator or after the 'as' keyword.

Closes PyCQA#341
This fixes issue PyCQA#518, pep8 still referenced in the cli help command
As a side effect, `[pep8]` in setup.cfg now becomes `[pycodestyle]`
Also, changed the path for the config file from ~/.config/pep8 to ~/.config/pycodestyle
These feel like changes that should have come with the jump to version 2.0.0, as they are breaking,
but support for  as a name can still be added if it's desired enough
Change all references to the pep8 project to say pycodestyle
Update Python version list in docs
Based on feedback on pull request PyCQA#534 on GitHub

Closes PyCQA#534
This test passes currently, but would need to be updated
if the functionality changes.
Detect single letter variable names enumerated in PEP-0008
This also updates the Checker to update it's state for physical lines
as well as logical lines. This allows maximum_line_length to rely on
Checker state for its noqa check.

Closes PyCQA#538
Neither the warnings modules, nor the two local assignments in
pycodestyle's newest check were being used. This removes them and
fixes pyflakes errors raised as a result.
Remove unused imports and assignments
Update maximum_line_length to use Checker.noqa
daspecster and others added 29 commits September 18, 2016 16:14
…-config

Check for both [pep8] and [pycodestyle] config sections.
Fix E305 regression in 2.1.0 due to fix for PyCQA#400
Prevent lines like 

    classification_error = 0 

From becoming treated as a class.
First step on getting PyCQA#206 merged into master.
Next steps:
 - New tests for strange indents
 - Fix help string for  to be more useful to users
 - Fix the documentation to account for the new cli input
Test that setting the indent_size to something other than 4 actually works and gives us correct Errors
(Related to a comment on the original pull request)
Also, updated docs/intro.rst with the new command
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.