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

pep257 use config file #124

Closed
jakirkham opened this issue May 15, 2015 · 5 comments
Closed

pep257 use config file #124

jakirkham opened this issue May 15, 2015 · 5 comments

Comments

@jakirkham
Copy link
Contributor

Much like pep8, pep257 allows for configuration to be supplied through config file (e.g. setup.cfg, tox.ini, or .pep257). However, this functionality does not appear to be supported in prospector at present. It would be nice to see this added. If this is possible and I am missing something, please help me understand how to go about doing this.

@tomato42
Copy link

tomato42 commented Nov 5, 2015

+1 lack of support for this (and apparent override by prospector of the default config in pep257) cause it to regress the PyCQA/pydocstyle#141 issue

@tomato42
Copy link

tomato42 commented Nov 5, 2015

that being said, it is possible to change pep257 configuration when run under prospector. In .landscape.yaml add following section to disable e.g. error D211:

pep257:
    disable:
        - D211

see documentation here: https://docs.landscape.io/advanced.html#existing-configuration

@jakirkham
Copy link
Contributor Author

Friendly nudge. Any news on this?

@carlio carlio closed this as completed Jul 22, 2018
@jakirkham
Copy link
Contributor Author

Was there a fix somewhere for this issue?

@FraBle
Copy link

FraBle commented Feb 28, 2019

Has this ever been fixed?
What is the current approach to fix the conflict between D203 and D211 as mentioned above (PyCQA/pydocstyle#141) by @tomato42

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

4 participants