Skip to content

Migrate packaging from setup.py to pyproject.toml#92

Merged
cokelaer merged 2 commits intomainfrom
copilot/update-to-pyproject-format
Mar 6, 2026
Merged

Migrate packaging from setup.py to pyproject.toml#92
cokelaer merged 2 commits intomainfrom
copilot/update-to-pyproject-format

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 5, 2026

Replaces the monolithic setup.py with a pyproject.toml (PEP 517/518/621), moving all package metadata and configuration to the standard modern format.

Changes

  • pyproject.toml (new): Full package metadata — name, version, description, authors, license, classifiers, dependencies, optional extras (testing, doc), src-layout package discovery, package data, and pytest config ([tool.pytest.ini_options])
  • setup.py (stripped): Reduced to only the C extension definition (spectrum.mydpss), which setuptools still requires in setup.py
  • .github/workflows/pypi.yml: Replaced deprecated python setup.py sdist with python -m build

Cleanup

  • Removed EOL Python version classifiers (2.7, 3.5, 3.7) that conflicted with requires-python = ">=3.8"
  • Removed redundant plot optional-dependency group (matplotlib is already a core dependency)

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Use pyproject instead of setup.py Migrate packaging from setup.py to pyproject.toml Mar 5, 2026
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 97.016%. remained the same
when pulling d8cbc2f on copilot/update-to-pyproject-format
into c910058 on main.

@cokelaer cokelaer marked this pull request as ready for review March 6, 2026 13:51
@cokelaer cokelaer merged commit abf5b76 into main Mar 6, 2026
6 of 7 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.

3 participants