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

Aggregate all sorts of tools configurations in pyproject.toml #1072

Closed
fwitte opened this issue May 30, 2024 · 1 comment · Fixed by #1073
Closed

Aggregate all sorts of tools configurations in pyproject.toml #1072

fwitte opened this issue May 30, 2024 · 1 comment · Fixed by #1073

Comments

@fwitte
Copy link
Member

fwitte commented May 30, 2024

Having many configuration files (many of the files starting with . in the root) as well as the setup.cfg, setup.py and pyproject.toml at the same time really increases maintenance effort, since you always have to search multiple files to find what you are trying to change. Therefore, I suggest

  • we move everything possible into pyproject.toml,
  • delete setup.py and setup.cfg and
  • put all configurations not available for pyproject.toml which were formerly in setup.cfg into their respective configuration files e.g. .flake8 as it seems

On top of that, I would like to critically review, what of all those things is actually required and take out the trash. E.g. the pep257 thing in setup.cfg seems ancient.

@fwitte
Copy link
Member Author

fwitte commented May 30, 2024

For example, there are three configurations for the same thing:

.prospector.yml, [pydocstyle] and [pep257] (both in setup.cfg. According to http://www.pydocstyle.org/en/stable/usage.html#configuration-files this can be handled by pyproject.toml.

p-snft added a commit that referenced this issue May 31, 2024
…ration-files

Merge configurations into `pyproject.toml`
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

Successfully merging a pull request may close this issue.

1 participant