Skip to content

Conversation

@jameslamb
Copy link
Member

Bundling together a couple of small maintenance updates.

Switch to SPDX license identifiers

The most important update, because without it CI will start breaking here in a few months.

Resolves warnings about license metadata by switching to an SPDX identifier and moving the floor on setuptools to >=77.0.0.

logs with relevant warnings (click me)
* Building wheel...
/tmp/build-env-bxde7cxp/lib/python3.9/site-packages/setuptools/config/_apply_pyprojecttoml.py:82: SetuptoolsDeprecationWarning: `project.license` as a TOML table is deprecated
!!

        ********************************************************************************
        Please use a simple string containing a SPDX expression for `project.license`. You can also use `project.license-files`. (Both options available on setuptools>=77.0.0).

        By 2026-Feb-18, you need to update your project and remove deprecated calls
        or your builds will no longer be supported.

        See https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#license for details.
        ********************************************************************************

!!
  corresp(dist, value, root_dir)
/tmp/build-env-bxde7cxp/lib/python3.9/site-packages/setuptools/config/_apply_pyprojecttoml.py:61: SetuptoolsDeprecationWarning: License classifiers are deprecated.
!!

        ********************************************************************************
        Please consider removing the following classifiers in favor of a SPDX license expression:

        License :: OSI Approved :: Apache Software License

        See https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#license for details.
        ********************************************************************************

!!
  dist._finalize_license_expression()
/tmp/build-env-bxde7cxp/lib/python3.9/site-packages/setuptools/dist.py:759: SetuptoolsDeprecationWarning: License classifiers are deprecated.
!!

        ********************************************************************************
        Please consider removing the following classifiers in favor of a SPDX license expression:

        License :: OSI Approved :: Apache Software License

        See https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#license for details.
        ********************************************************************************

!!
  self._finalize_license_expression()

(build link)

Establish CODEOWNERS

Proposes making @rapidsai/packaging-codeowners CODEOWNERS for this repo, so we'll get auto-assigned reviewers.

Update all pre-commit hooks

It's been a while, so I ran this:

pre-commit autoupdate

Alphabetize and reformat [tool.ruff.lint] table in pyproject.toml

Because we're using a CI run anyway and because formatting it like that makes it a bit easier to read (especially in git diff form) as the list of ruff rules grows to something like this: https://github.com/rapidsai/cudf/blob/9ebb19b30e6dbfae7921cb023552b5095e39a8f4/pyproject.toml#L51

@@ -0,0 +1 @@
* @rapidsai/packaging-codeowners
Copy link
Member Author

@jameslamb jameslamb Dec 4, 2025

Choose a reason for hiding this comment

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

TODO before merging:

  • agree on the rules here
  • get any necessary teams added as contributors

Copy link
Contributor

Choose a reason for hiding this comment

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

Happy to see this change. +1!

@@ -0,0 +1 @@
* @rapidsai/packaging-codeowners
Copy link
Contributor

Choose a reason for hiding this comment

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

Happy to see this change. +1!

Copy link

@gforsyth gforsyth left a comment

Choose a reason for hiding this comment

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

+1 from me. I'm going to merge this then configure the review assignment

@gforsyth gforsyth merged commit fea49a0 into main Dec 5, 2025
4 checks passed
@gforsyth gforsyth deleted the misc-maintenance-updates branch December 5, 2025 13:56
GPUtester pushed a commit that referenced this pull request Dec 5, 2025
## [1.20.2](v1.20.1...v1.20.2) (2025-12-05)

### Bug Fixes

* **packaging:** use SPDX license config, other small maintenance ([#172](#172)) ([fea49a0](fea49a0))
@GPUtester
Copy link

🎉 This PR is included in version 1.20.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants