Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

Show linter codes #28

Closed
bmhkim opened this issue Mar 12, 2015 · 10 comments
Closed

Show linter codes #28

bmhkim opened this issue Mar 12, 2015 · 10 comments

Comments

@bmhkim
Copy link

bmhkim commented Mar 12, 2015

I'd really like to see the linter code included with the linter message. An example use case involves working on a codebase that's not entirely PEP-8 compliant. (e.g. tabs instead of spaces) Display of the pylint code makes it easier to disable the message in the project's .pylintrc.

@posita
Copy link

posita commented May 2, 2015

Please include the human readable versions as well (e.g., undefined-variable is synonymous with E0602, etc.).

@bmhkim
Copy link
Author

bmhkim commented Jun 26, 2015

I've got a fix for this hacked together, and will submit a pull request.

@bmhkim
Copy link
Author

bmhkim commented Jun 26, 2015

@posita If you want this immediately, I have a fork that seems to be working for me: bmhkim/linter-pylint

@posita
Copy link

posita commented Jun 30, 2015

Nice! 👍

@SpainTrain
Copy link
Member

👍

@SpainTrain
Copy link
Member

This should be closed in the latest released version. @bmhkim mind confirming and closing if true?

@posita
Copy link

posita commented Aug 31, 2015

LGTM:

screen shot 2015-08-31 at 08 39 48

@SpainTrain
Copy link
Member

@posita are those import errors true positives? There is an issue open for false positives on relative imports (#58).

Thanks for testing!

@posita
Copy link

posita commented Aug 31, 2015

@posita are those import errors true positives?

They are valid(ish). They are absolute imports provided via a virtualenv that Atom is not aware of, so I'm pretty sure they are not implicated by #58.

@SpainTrain
Copy link
Member

Awesome, just wanted to make sure =-D

Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants