Skip to content

Issue 366#367

Merged
IanLee1521 merged 3 commits intomasterfrom
issue-366
Jan 12, 2015
Merged

Issue 366#367
IanLee1521 merged 3 commits intomasterfrom
issue-366

Conversation

@IanLee1521
Copy link
Member

I added a few test cases to the file testsuite/e30not.py. Please let me know if I missed any cases that this change doesn't cover. Resolves #366

pep8 should ignore whitespace errors E30 when evaluating non-top level code.
Move checking of def / class to higher spot in if statement.
This changes some E303 errors into E302 errors, such as for:

    def a():\n    pass\n\n\n\ndef b():\n    pass
IanLee1521 added a commit that referenced this pull request Jan 12, 2015
@IanLee1521 IanLee1521 merged commit ff1d2b7 into master Jan 12, 2015
@IanLee1521 IanLee1521 deleted the issue-366 branch January 13, 2015 07:59
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.

E303: Does not report, that a blank line between classes is needed, if they're defined inside an if statement

1 participant