Skip to content

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

Merged
rapids-bot[bot] merged 4 commits intorapidsai:branch-25.04from
jameslamb:update-dependencies-yaml
Feb 4, 2025
Merged

raise setuptools floor to 61.0.0, sphinx floor to 8.0.0, other small dependencies cleanup#820
rapids-bot[bot] merged 4 commits intorapidsai:branch-25.04from
jameslamb:update-dependencies-yaml

Conversation

@jameslamb
Copy link
Copy Markdown
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-pr-bot
Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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! 🙏

Comment thread dependencies.yaml
# 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
Copy Markdown
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
Copy Markdown
Member

/merge

@rapids-bot rapids-bot Bot merged commit 08a6bc2 into rapidsai:branch-25.04 Feb 4, 2025
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