Skip to content

Migrate to PEP 735 dependency groups #91

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

Closed
ncoghlan opened this issue Nov 25, 2024 · 1 comment · Fixed by #121
Closed

Migrate to PEP 735 dependency groups #91

ncoghlan opened this issue Nov 25, 2024 · 1 comment · Fixed by #121
Labels
Category: Maintainability Proposed improvement to project maintainability CI (Automation) Continuous integration (and other automation) good first issue Good for newcomers

Comments

@ncoghlan
Copy link
Collaborator

pdm 2.20.1+ supports PEP 735 dependency groups, which will allow migrating away from the PDM-specific development dependency declarations.

tox 4.22.0 also supports PEP 735 dependency groups: tox-dev/tox@568b90f

After migration, tox-pdm shouldn't be necessary anymore (since we're just using it to install specific PDM dependency groups in particular environments)

@ncoghlan ncoghlan added good first issue Good for newcomers CI (Automation) Continuous integration (and other automation) Category: Maintainability Proposed improvement to project maintainability labels Nov 25, 2024
ncoghlan added a commit that referenced this issue Apr 2, 2025
@ncoghlan
Copy link
Collaborator Author

ncoghlan commented Apr 2, 2025

#121 migrates to the standardised dependency group format (and takes advantage of group inclusion to make the purpose of the various entries in the "dev" group clearer)

tox-pdm remains as a dependency, as it turned out to be serving a second purpose: ensuring the tox environments respect the PDM lock file version pins.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Maintainability Proposed improvement to project maintainability CI (Automation) Continuous integration (and other automation) good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant