Skip to content

Move static metadata from setup.py/setup.cfg into pyproject.toml - #2163

Open
jjardon wants to merge 2 commits into
apache:masterfrom
jjardon:jjardon/modernize_project_toml
Open

Move static metadata from setup.py/setup.cfg into pyproject.toml#2163
jjardon wants to merge 2 commits into
apache:masterfrom
jjardon:jjardon/modernize_project_toml

Conversation

@jjardon

@jjardon jjardon commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Move the static project metadata that can be expressed with PEP 621 into the [project] table of pyproject.toml and remove the corresponding duplicated arguments from setup() and the python_requires line from setup.cfg.

@abderrahim

Copy link
Copy Markdown
Contributor

Looks like this somehow changed the version of black used

Comment thread pyproject.toml Outdated
@jjardon
jjardon force-pushed the jjardon/modernize_project_toml branch 2 times, most recently from 958e2ab to 86f60ac Compare August 7, 2026 13:25
jjardon added 2 commits August 7, 2026 14:27
Move the static project metadata that can be expressed with PEP 621
into the [project] table of pyproject.toml and remove the corresponding
duplicated arguments from setup() and the python_requires line from
setup.cfg.
@jjardon
jjardon force-pushed the jjardon/modernize_project_toml branch from 86f60ac to 7770a15 Compare August 7, 2026 13:27
@juergbi

juergbi commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Looks like this somehow changed the version of black used

It's not a different version of black. The behavior of black is affected by requires-python in pyproject.toml (but apparently not by the equivalent in setup.cfg, which we had before).

@juergbi

juergbi commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

I've opened #2164 to separate the partial reformatting from this branch. The black issue should disappear when rebasing this branch on top of #2164.

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 this pull request may close these issues.

3 participants