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

Landscape needs a documented way to set global PyLint rules #56

Closed
ssokolow opened this issue Jul 4, 2014 · 2 comments
Closed

Landscape needs a documented way to set global PyLint rules #56

ssokolow opened this issue Jul 4, 2014 · 2 comments

Comments

@ssokolow
Copy link

ssokolow commented Jul 4, 2014

It feel as if every other line in my D-Bus code consists of the comment

# pylint: disable=bad-continuation

...because the only human-readable alternative would be to make ridiculously long lines, rely on word-wrap, and leverage Landscape's support for shutting up PyFlakes instead.

@pennyarcade
Copy link

👍 :)
If you're looking for a ".landscape.yaml" in the root dir why not look for a ".pylintrc" in the format "pylint --generate-rcfile" gives you. This makes it really easy for users to get started.
(I already have one for my shippable.com builds)

That shouldn't be hard to implement since you already have the infrastructure there and you only have to hand it to pylint as is.

You can expand this pattern without much inconvenience for the other tools. Only (small) drawback I see is that you will end up with 3-5 config files in your projects root for landscape alone plus some more for other services.
I already have 4 :)

@carlio
Copy link
Member

carlio commented Apr 3, 2015

Finally sorted out a proper documentation site: https://docs.landscape.io/suppressing.html

if a .pyintrc exists it will be used by Prospector (and I should document that better)

@carlio carlio closed this as completed Apr 3, 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

3 participants