Skip to content

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

Merged
juergbi merged 1 commit into
apache:masterfrom
jjardon:jjardon/modernize_project_toml
Aug 28, 2026
Merged

Move static metadata from setup.py/setup.cfg into pyproject.toml#2163
juergbi merged 1 commit 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 3 times, most recently 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.

Comment thread pyproject.toml Outdated
Comment thread pyproject.toml
Comment thread setup.cfg
@jjardon
jjardon force-pushed the jjardon/modernize_project_toml branch from e81229b to 547b15f Compare August 27, 2026 16:55
Comment thread tox.ini Outdated
@jjardon
jjardon force-pushed the jjardon/modernize_project_toml branch 2 times, most recently from 80dd621 to 64b7716 Compare August 27, 2026 17:40
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 64b7716 to e86ce4f Compare August 27, 2026 17:58
@juergbi
juergbi merged commit 4bd905b into apache:master Aug 28, 2026
17 checks passed
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.

4 participants