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

config not being loaded from ~/.config/pep8 #245

Closed
mcandre opened this issue Nov 10, 2013 · 6 comments
Closed

config not being loaded from ~/.config/pep8 #245

mcandre opened this issue Nov 10, 2013 · 6 comments

Comments

@mcandre
Copy link

mcandre commented Nov 10, 2013

I followed the official documentation for configuration, placing my settings in ~/.config/pep8, in INI format.

When I put a tox.ini at my project root directory, pep8 successfully loads my custom configuration. But when I move the same settings to ~/.config/pep8, pep8 ignores them.

@noorul
Copy link

noorul commented Nov 16, 2013

I am facing another problem could be related.

I have .ropeproject folder under project root folder which is specific to a software that I use. I was looking for options to make pep8 to exclude it. Adding it to ~/.config/pep8 is not helping as this is overwritten by tox.ini settings. As you can see pep8 doesn't care about global settings if local settings exists. https://github.com/jcrocholl/pep8/blob/master/pep8.py#L1760

@kynan
Copy link
Contributor

kynan commented Dec 3, 2013

That's not quite true: ~/.config/pep8 is read first, followed by any project specific settings in tox.ini or setup.cfg.

Note that you still need to add the [pep8] section heading.

@noorul
Copy link

noorul commented Dec 3, 2013

@kynan Is your response to my problem?

@kynan
Copy link
Contributor

kynan commented Dec 3, 2013

@noorul both yours and @mcandre's.

@florentx
Copy link
Contributor

I would need more information about the case which does not work.

The user configuration is merged with the project configuration at runtime, but if an option is present in both files, the project configuration takes precedence.

@IanLee1521
Copy link
Member

Closing as there has been no update to this issue in almost 2 years.

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

No branches or pull requests

5 participants