Skip to content
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

"disable" yaml for pep8 is not honored #97

Closed
jquast opened this issue Jan 10, 2015 · 1 comment
Closed

"disable" yaml for pep8 is not honored #97

jquast opened this issue Jan 10, 2015 · 1 comment

Comments

@jquast
Copy link
Contributor

jquast commented Jan 10, 2015

.prospector.yaml:

pep8:
    run: true
    disable:
        - N802

test.py:

def MyFunc():
    pass
$ prospector --tool=pep8 test.py
Messages
========

test.py
  Line: 1
    pep8: N802 / function name should be lowercase (col 5)



Check Information
=================
       Started: 2015-01-09 18:05:50.651303
      Finished: 2015-01-09 18:05:50.653521
    Time Taken: 0.00 seconds
     Formatter: grouped
    Strictness: from profile
Libraries Used:
     Tools Run: pep8
 Message Found: 1

seems to work fine for pylint, and frosted, however.

@carlio
Copy link
Contributor

carlio commented Jan 10, 2015

Not quite sure when this started happening, but #96 fixes it.

@carlio carlio closed this as completed Jan 10, 2015
carlio added a commit to carlio/prospector that referenced this issue Jan 10, 2015
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

No branches or pull requests

2 participants