Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

raise setuptools floor to 61.0.0, sphinx floor to 8.0.0, other small dependencies cleanup #820

Merged

Conversation

jameslamb
Copy link
Member

@jameslamb jameslamb commented Jan 31, 2025

Contributes to rapidsai/build-planning#146

  • updates setuptools dependency to >=61.0.0 (the first version to support pyproject.toml)
  • makes numpydoc pins in testing and docs the same
  • removes ceiling on sphinx, changes sphinx dependency to >=8.0.0 (matching what was recently done across the RAPIDS, e.g. in Remove sphinx pinning cuml#6195)
  • ensures numpydoc makes it into requirements output from rapids-dependency-file-generator

@jameslamb jameslamb added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Jan 31, 2025
Copy link

copy-pr-bot bot commented Jan 31, 2025

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@jameslamb
Copy link
Member Author

/ok to test

@jameslamb jameslamb changed the title WIP: raise setuptools floor to 61.0.0, other small dependencies cleanup WIP: raise setuptools floor to 61.0.0, sphinx floor to 8.0.0, other small dependencies cleanup Feb 3, 2025
@jameslamb jameslamb changed the title WIP: raise setuptools floor to 61.0.0, sphinx floor to 8.0.0, other small dependencies cleanup raise setuptools floor to 61.0.0, sphinx floor to 8.0.0, other small dependencies cleanup Feb 3, 2025
@jameslamb jameslamb marked this pull request as ready for review February 3, 2025 16:08
@jameslamb jameslamb requested a review from a team as a code owner February 3, 2025 16:08
Copy link
Member

@jakirkham jakirkham left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks James! 🙏

# TODO: sphinx upper version limit can likely be removed after pydata 0.13
# is released. See https://github.com/pydata/pydata-sphinx-theme/issues/1094
- sphinx<6
- sphinx>=8.0.0
Copy link
Member

Choose a reason for hiding this comment

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

As noted in the comment, this was pinned to an old version of sphinx in PR ( #481 ) to workaround this upstream pydata-sphinx-theme issue ( pydata/pydata-sphinx-theme#1094 )

Looks like that issue was subsequently fixed in PR ( pydata/pydata-sphinx-theme#1097 ) as part of pydata-sphinx-theme's 0.13.0 release

Given that has been out for a while (and several versions have since been released), think we are ok dropping it

@jakirkham
Copy link
Member

/merge

@rapids-bot rapids-bot bot merged commit 08a6bc2 into rapidsai:branch-25.04 Feb 4, 2025
45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improves an existing functionality non-breaking Introduces a non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants