Skip to content

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

Merged
janhoy merged 8 commits into
apache:mainfrom
janhoy:migrate-to-pyproject-toml
May 25, 2026
Merged

Migrate packaging from setup.py to pyproject.toml#22
janhoy merged 8 commits into
apache:mainfrom
janhoy:migrate-to-pyproject-toml

Conversation

@janhoy
Copy link
Copy Markdown
Contributor

@janhoy janhoy commented May 23, 2026

Closes #16

Replaces setup.py, setup.cfg, and tox.ini with a single pyproject.toml containing all package metadata, dependencies, and tool configuration (pytest, tox).

All 1090 unit tests pass.

This comment was marked as resolved.

- Fix MANIFEST.in: include min-version.txt (not min-os-version.txt)
- Upgrade pytest-benchmark to >=4.0.0 to fix py.io.TerminalWriter error on Python 3.12
- Update PYTHON_SUPPORT_GUIDE.md: replace setup.py/tox.ini refs with pyproject.toml
- Makefile: enforce tox>=4 when installing tox for it/it312/it313 targets
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 5 comments.

Comment thread Makefile
Comment thread Makefile
Comment thread PYTHON_SUPPORT_GUIDE.md Outdated
Comment thread pyproject.toml
Comment thread pyproject.toml
janhoy added 4 commits May 24, 2026 21:49
- .coveragerc: remove setup.py from coverage omit list (file no longer exists)
- run.sh: rename install_osbenchmark_with_setuptools -> install_osbenchmark
  and remove misleading setup.py comment (function uses pip install, not setup.py)
pyproject.toml [tool.tox] config requires tox 4+. The previous check
only installed tox when missing; now it also upgrades if the installed
version is older than 4.
…GUIDE.md

The .github/workflows/integ-tests.yml file does not exist in this repo.
Python version support is driven solely by .ci/variables.json and pyproject.toml.
pytest-benchmark 3.x uses py.io.TerminalWriter which was removed from the
py library in 1.11+, causing an INTERNALERROR on Python 3.12. The upgrade
to >=4.0.0 is intentional and necessary for Python 3.12 compatibility.
@janhoy janhoy requested a review from epugh May 24, 2026 19:57
Copy link
Copy Markdown

@epugh epugh left a comment

Choose a reason for hiding this comment

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

Sure, I guess this is the direction folks are going in ;-)

@janhoy janhoy merged commit 7d7455f into apache:main May 25, 2026
3 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.

Migrate packaging from setup.py to pyproject.toml

3 participants