Skip to content

Pycodestyle update causing our lint to fail. #2974

Closed

Description

@daspecster

From what I can tell, there are three new errors that pycodestyle is finding.

E306: expected 1 blank line before a nested definition, found 0
E123: closing bracket does not match indentation of opening bracket's line
E722: do not use bare except

See: #2973

Activity

  1. self-assigned this
    on Jan 30, 2017
  2. dhermes commented on Jan 30, 2017

    @dhermes
    Contributor

    E123 seems legit

  3. daspecster commented on Jan 30, 2017

    @daspecster
    ContributorAuthor

    Yeah, I updated the PR to just fix the E123 errors.

  4. dhermes commented on Jan 30, 2017

    @dhermes
    Contributor
  5. dhermes commented on Jan 31, 2017

    @dhermes
    Contributor

    They fixed it in 2.3.1

  6. dhermes commented on Jan 31, 2017

    @dhermes
    Contributor

    @daspecster I should've been more specific, they fixed the E306 false positives. We still have the E722 and maybe the E123 issues to deal with

  7. daspecster commented on Jan 31, 2017

    @daspecster
    ContributorAuthor

    Yeah I saw, maybe closed this too soon, but the rest are resolved in #2973 either way. I have an update to remove the ignore for E306 then it should be good IMO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions