Skip to content

Switch to pyproject.toml, other tooling updates#152

Merged
mattwthompson merged 8 commits into
mainfrom
copilot/switch-to-pyproject-toml
May 21, 2026
Merged

Switch to pyproject.toml, other tooling updates#152
mattwthompson merged 8 commits into
mainfrom
copilot/switch-to-pyproject-toml

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 27, 2026

I took the bot changes, made them actually work, and also

  • Switched to setuptools_scm, which has the benefits of
    • Not being vendored code that easily falls out of date
    • Does not require the full git history
    • Works for me locally
  • Added pyproject-fmt linter
  • Updated the docstring of get_forcefield_dirs_paths

Bot text below the line

Consolidates all Python packaging and tooling config into pyproject.toml, removing setup.py, setup.cfg, and MANIFEST.in.

Description

  • [build-system]setuptools + versioneer[toml]
  • [project] — metadata (name, description, authors, license, readme, entry points, package data) migrated from setup.py
  • [tool.versioneer] — moved from [versioneer] in setup.cfg
  • [tool.coverage.run] — moved from [coverage:run] in setup.cfg
  • Deleted: setup.py, setup.cfg, MANIFEST.in

Copilot AI linked an issue Apr 27, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Update Python configuration to use pyproject.toml Switch to pyproject.toml Apr 27, 2026
Copilot AI requested a review from mattwthompson April 27, 2026 13:43
* Update `pyproject.toml`
* Switch to `setuptools_scm`
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (edd7724) to head (32ad142).

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mattwthompson mattwthompson marked this pull request as ready for review April 27, 2026 14:06
@mattwthompson mattwthompson requested a review from j-wags as a code owner April 27, 2026 14:06
@mattwthompson mattwthompson mentioned this pull request Apr 27, 2026
@j-wags j-wags assigned j-wags and unassigned mattwthompson May 20, 2026
Copy link
Copy Markdown
Member

@j-wags j-wags left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The mentions of versioneer here are wrong, I think, since it was fully removed:

Description
[build-system] — setuptools + versioneer[toml]
[project] — metadata (name, description, authors, license, readme, entry points, package data) migrated from setup.py
[tool.versioneer] — moved from [versioneer] in setup.cfg
[tool.coverage.run] — moved from [coverage:run] in setup.cfg

Other than that this looks good - I installed and tested locally and was able to list installed FFs and get the package version. Thanks @mattwthompson!

@mattwthompson
Copy link
Copy Markdown
Member

Yep that comment is out of date from changes I made after Copilot worked

I poked through logs for warnings and didn't see anything. Will merge when green. Thanks!

@mattwthompson mattwthompson changed the title Switch to pyproject.toml Switch to pyproject.toml, other tooling updates May 21, 2026
@mattwthompson mattwthompson merged commit 46c1741 into main May 21, 2026
13 checks passed
@j-wags j-wags assigned mattwthompson and unassigned j-wags May 21, 2026
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.

Switch to pyproject.toml

4 participants