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

Allow tools to use exsting configuration if found or provided. #74

Closed
carlio opened this issue Nov 24, 2014 · 3 comments
Closed

Allow tools to use exsting configuration if found or provided. #74

carlio opened this issue Nov 24, 2014 · 3 comments

Comments

@carlio
Copy link
Member

carlio commented Nov 24, 2014

For projects which already have configuration for various tools, prospector should use and prefer that. This is a more general version of #10.

This already happens for pep8, however, only if the files are in specified positions. There should be a command line option (--config-for-pep8=some/path) which allows extra places to be searched.

@mbertolacci
Copy link

This would be very useful for pylint because I would like to import our own pylint plugin. Or is there another way to do that?

carlio added a commit that referenced this issue Jan 2, 2015
…emove the "adaptors" concept which was confusing and not particularly useful.
carlio added a commit that referenced this issue Jan 2, 2015
… new configuration manager introduced in #74 and #10
@carlio
Copy link
Member Author

carlio commented Jan 2, 2015

Prospector will now use a .pylintrc file or pylintrc file if one is found in the directory that prospector is run from. If you need to, you can also explicitly specify it like:

prospector --pylint-config-file path/to/my/pylintrc

Note that if this is used, prospector's own opinions on what should be enabled/disabled are completely ignored!

@carlio
Copy link
Member Author

carlio commented Jan 2, 2015

@mbertolacci This should enable you to import plugins, let me know if it doesn't work!

@carlio carlio closed this as completed Jan 4, 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