GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Move project metadata to pyproject.toml by mwtoews · Pull Request #2242 · gitpython-developers/GitPython

Conversation

@mwtoews

@mwtoews mwtoews commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

This PR is a near 1:1 conversion of metadata from setup.py to pyproject.toml, as supported by setuptools. There are a few differences from the legacy forms:

  • Multiple authors and emails are listed
  • Multiple URLs are listed
  • A keyword "git" is added

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

EliahKagan reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
@mwtoews
mwtoews force-pushed the pyproject-toml branch 2 times, most recently from 295b922 to 3d601f5 Compare September 16, 2026 10:39
@Byron

Byron commented Sep 16, 2026

Copy link
Copy Markdown
Member

Thanks a lot, I am definitely interested in mostly getting rid of setup.py. Once CI passes, I think this should be merged and if there is trouble with the next publish, it's probably an easy fix, too.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@Byron

Byron commented Sep 16, 2026

Copy link
Copy Markdown
Member

Something to consider:

  • [P2] Raise the setuptools minimum to 62.6 — /Users/byron/dev/github.com/gitpython-developers/GitPython.pyproject-toml/pyproject.toml:2-2
    Setuptools 61.x–62.5.x satisfies this requirement but cannot validate the new tool.setuptools.dynamic.dependencies and optional-dependencies entries: file-based dependency directives were introduced in 62.6.0. Source builds using those older backends therefore fail despite meeting the declared build requirements. Raise the lower bound to setuptools >= 62.6.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@mwtoews

mwtoews commented Sep 16, 2026

Copy link
Copy Markdown
Contributor Author

The failing test with Python 3.7 is test/test_installation.py::TestInstallation::test_installation with message

setuptools.config.pyprojecttoml._BetaConfiguration: Support for [tool.setuptools] in pyproject.toml is still beta.

This was true at the time (for that version of setuptools) but not today, so PYTHONWARNINGS is set to "default" for Python 3.7.

The setup.py file might be needed until Python 3.9 is the minimum, as it needs to work-around setuptools deprecation messages described here. The workaround is to set "licence" as dynamic, even though it isn't.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@mwtoews
Move project metadata to pyproject.toml
b7d8f1a
@mwtoews
mwtoews force-pushed the pyproject-toml branch from 3d601f5 to b7d8f1a Compare September 16, 2026 10:46
@mwtoews

mwtoews commented Sep 16, 2026

Copy link
Copy Markdown
Contributor Author

Something to consider ... Raise the lower bound to setuptools >= 62.6.

Done.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@Byron
Byron merged commit 7609492 into gitpython-developers:main Sep 16, 2026
50 checks passed

Uh oh!

There was an error while loading. Please reload this page.

@mwtoews
mwtoews deleted the pyproject-toml branch September 16, 2026 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@mwtoews @Byron
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.