Skip to content

MNT: unify static configuration files into pyproject.toml, update PEP 639 license metadata#371

Merged
jswhit merged 3 commits into
Unidata:masterfrom
neutrinoceros:mnt/pyproject-all
Sep 23, 2025
Merged

MNT: unify static configuration files into pyproject.toml, update PEP 639 license metadata#371
jswhit merged 3 commits into
Unidata:masterfrom
neutrinoceros:mnt/pyproject-all

Conversation

@neutrinoceros
Copy link
Copy Markdown
Contributor

In preparation for a PR in the spirit of Unidata/netcdf4-python#1427, modernize the project configuration around pyproject.toml

@neutrinoceros neutrinoceros force-pushed the mnt/pyproject-all branch 4 times, most recently from fd7f385 to 2b9ae0e Compare September 16, 2025 08:21
Comment thread setup.py
from Cython.Build import cythonize
from setuptools import Command, Extension, setup

# https://github.com/Unidata/cftime/issues/34
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

note: direct invocations of setup.py as a scripts are deprecated in setuptools, so gh-34 should not be an issue now.

Comment thread pyproject.toml
requires-python = ">=3.8"

[dependency-groups]
dev = [
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

note that I redefined requirements-tests.txt as a PEP 735 dependency group, which accomplishes the same thing: dev-only, and user-invisible dependencies. This PEP is already supported by pip, uv, cibuildwheel, tox and (I suspect) pdm.

@neutrinoceros
Copy link
Copy Markdown
Contributor Author

I was able to reproduce and fix the failure locally. Hopefully this should be good to go now, unless I broke something else specifically in CI configuration.

@neutrinoceros
Copy link
Copy Markdown
Contributor Author

neutrinoceros commented Sep 23, 2025

I see the build timed-out on manylinux+aarch64 (which is done via emulation). However it's not trivial to guess whether this PR has anything to do with it, given the last successful run was more than 6 months ago and its logs are long gone.

@jswhit jswhit merged commit 312b8d7 into Unidata:master Sep 23, 2025
34 of 35 checks passed
@neutrinoceros neutrinoceros deleted the mnt/pyproject-all branch September 24, 2025 04:40
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.

2 participants