Skip to content

Modernize pyproject.toml to work with uv#200

Merged
drasmuss merged 3 commits intomainfrom
pyproject-update
Mar 18, 2025
Merged

Modernize pyproject.toml to work with uv#200
drasmuss merged 3 commits intomainfrom
pyproject-update

Conversation

@hunse
Copy link
Contributor

@hunse hunse commented Mar 6, 2025

Motivation and context:

This is required to be able to do uv pip install -e . and create an editable install.

It modernizes the pyproject.toml file according to the following primer:
See https://packaging.python.org/en/latest/guides/modernize-setup-py-project/#modernize-setup-py-project

NOTE: This is a "minimal" modernization to make us standards-compliant and get things working with uv. Moving all non-programmatic metadata (e.g. project name, author, etc.) from setup.py to pyproject.toml is recommended, and we could pursue this at some point.

How has this been tested?

Installed it locally and it works.

How long should this take to review?

  • Quick (less than 40 lines changed or changes are straightforward)

Types of changes:

  • New feature (non-breaking change which adds functionality)

Checklist:

  • I have read the CONTRIBUTING.rst document.
  • I have updated the documentation accordingly.
  • I have included a changelog entry.
  • I have added tests to cover my changes.
  • I have run the test suite locally and all tests passed.

Still to do:

@github-actions
Copy link

github-actions bot commented Mar 6, 2025

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  nengo_bones
  version.py
Project Total  

This report was generated by python-coverage-comment-action

@drasmuss drasmuss merged commit 2f10a6b into main Mar 18, 2025
9 checks passed
@drasmuss drasmuss deleted the pyproject-update branch March 18, 2025 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants