-
Notifications
You must be signed in to change notification settings - Fork 102
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
Feature: pyproject.toml support for configuration #115
Comments
gitlint is a tool that can be used outside of the python community. Having to support only 1 configuration file ( |
Thanks for your interest :-) I agree with @sigmavirus24 on this (both from usability and maintenance perspectives). We already have a similar request in #82, which I've left open since There's a few times I've wished I had picked something else than ini to better support complex types such as dictionaries and lists, but I don't want cause friction for end-users by switching formats at this point. The main reason I had picked ini is because that's what tools such as pylint and flake8 used at the time (just learned that pylint does support |
I won't be doing this as of now, for the aforementioned reasons. Closing this issue. |
An example is a codespell, which is a spell check written by python, is also a tool that can be used outside of the python community. They have added support to pyproject.toml and still keep support for
I agree One: every tool has its configuration. codespell use Two: flake8 can use Which is more confusing for users? Which is more convenient for users? Why not let user simply use one So I cannot agree |
No. No it doesn't. (I'm a maintainer of Flake8, I know.) @Freed-Wu I can't tell if you're just trying to make an argument and you don't bother to fact check yourself, or if you're intentionally lying to make a compelling case but much of what you've been saying has been incorrect. I won't bother to check
No, again, it provides affordances, not direction. It also doesn't bind the larger community to do anything. |
I just want to take an imaginary. In fact,
See https://github.com/codespell-project/codespell#using-a-config-file (In fact, their support is added recently) Using a config file
-------------------
Command line options can also be specified in a config file.
When running ``codespell``, it will check in the current directory for a file
named ``setup.cfg`` or ``.codespellrc`` (or a file specified via ``--config``),
containing an entry named ``[codespell]``. Each command line argument can
be specified in this file (without the preceding dashes), for example::
[codespell]
skip = *.po,*.ts,./src/3rdParty,./src/Test
count =
quiet-level = 3
Codespell will also check in the current directory for a ``pyproject.toml``
(or a path can be specified via ``--toml <filename>``) file, and the
``[tool.codespell]`` entry will be used as long as the tomli_ package
is installed, for example::
[tool.codespell]
skip = '*.po,*.ts,./src/3rdParty,./src/Test'
count = ''
quiet-level = 3
These are both equivalent to running::
codespell --quiet-level 3 --count --skip "*.po,*.ts,./src/3rdParty,./src/Test"
As a conclusion, it depends on you.
Depends on you, too.
OK. I have no intention to offend you. Now let us end up this discussion. |
With PEP518 and pyproject.toml becoming a standard way of configuring tools I was wondering it if it something that would be considered to be supported? I am happy to see if I can make the changes necessary. I wanted to get confirmation it is something that would be accepted before I put in the effort.
The text was updated successfully, but these errors were encountered: