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 configuration file #58

Closed
kdeldycke opened this issue Oct 7, 2013 · 14 comments
Closed

pep257 configuration file #58

kdeldycke opened this issue Oct 7, 2013 · 14 comments
Milestone

Comments

@kdeldycke
Copy link

@kdeldycke kdeldycke commented Oct 7, 2013

pep8 has ~/.config/pep8.
PyLint has ~/.pylintrc.
Flake8 has ~/.config/flake8.
Even PyChecker had ~/.pycheckrc.

Why can't pep257 have a configuration file ?

@keleshev
Copy link
Contributor

@keleshev keleshev commented Oct 7, 2013

Sure! Pull requests are welcome 😉

Loading

@kdeldycke
Copy link
Author

@kdeldycke kdeldycke commented Oct 11, 2013

I'll wait for the merging of your rewrite branch.

In the mean time, do you have any preferences on the naming and default location ?

Loading

@keleshev
Copy link
Contributor

@keleshev keleshev commented Oct 11, 2013

~/.config/pep257 should be best. Do you know if pep8/pylint/flake8/pychecker use any local (per-project) config files? That could be interesting.

Ini-style config is probably the best choice, since it's supported by standard library. Take a look at this config example. It allows for configs like:

[pep257]
--explain
--ignore=D101,D202
--match='(?!test_).*\.py'

Loading

@sigmavirus24
Copy link
Member

@sigmavirus24 sigmavirus24 commented Oct 11, 2013

We honor setup.cfg if I remember correctly. Possibly also tox.ini but I think that was deprecated. I'm also not sure if we support ~/.config/flake8 any longer.

[#]: We == Flake8

Loading

@keleshev
Copy link
Contributor

@keleshev keleshev commented Oct 11, 2013

What do you use if not ~/.config/flake8. setup.cfg and tox.ini are also good ideas.

Loading

@adiroiban
Copy link

@adiroiban adiroiban commented Apr 12, 2014

Before having a configuration file, I prefer to have PEP257Checker(options). options can be a simple key/value dict of Bunch like object.

Loading

@adiroiban
Copy link

@adiroiban adiroiban commented Apr 12, 2014

btw. I am not using (or plan to use) pep256 directly from the command line. Rather I load a file in memory once and then send the content to many linters and checkers.

Loading

@treyhunner
Copy link

@treyhunner treyhunner commented Jul 21, 2014

I would like to see setup.cfg support. I have particular files I would like to omit from checking and I don't want to type them at the command line ever time.

Loading

@Nurdok
Copy link
Member

@Nurdok Nurdok commented Jul 26, 2014

@kdeldycke are you working on this, or planning to?

Loading

@kdeldycke
Copy link
Author

@kdeldycke kdeldycke commented Jul 26, 2014

@Nurdok not at all. Feel free to take over this issue ! :)

Loading

@oryband
Copy link

@oryband oryband commented Dec 11, 2014

👍 any progress on this?

Loading

@Nurdok
Copy link
Member

@Nurdok Nurdok commented Dec 12, 2014

Not yet, but I consider this the next big thing.

On Thu, 11 Dec 2014 12:50 pm Ory Band notifications@github.com wrote:

[image: 👍] any progress on this?


Reply to this email directly or view it on GitHub
#58 (comment).

Loading

@Nurdok Nurdok added this to the 0.4.0 Release milestone Dec 31, 2014
@Nurdok Nurdok added this to the 0.4.0 Release milestone Dec 31, 2014
@Nurdok Nurdok closed this Jan 2, 2015
@oryband
Copy link

@oryband oryband commented Jan 2, 2015

Great. Could youp lease add documentation for this please? Specifically to the front page of this repo (README.md)

Loading

@sigmavirus24
Copy link
Member

@sigmavirus24 sigmavirus24 commented Jan 2, 2015

This is a better opportunity to start using readthedocs.org and have real documentation for this project.

Loading

@keleshev keleshev mentioned this issue Jan 2, 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
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
7 participants