Skip to content

expand CI-skipping logic, other small build changes#39

Merged
rapids-bot[bot] merged 1 commit intorapidsai:mainfrom
jameslamb:misc-build-updates
Jan 30, 2026
Merged

expand CI-skipping logic, other small build changes#39
rapids-bot[bot] merged 1 commit intorapidsai:mainfrom
jameslamb:misc-build-updates

Conversation

@jameslamb
Copy link
Copy Markdown
Member

Description

Proposes a batch of miscellaneous build / packaging / CI changes.

Changes

Expands CI-skipping logic

Contributes to rapidsai/build-planning#243

Tries to avoid unnecessary CI runs by making the CI-skipping rules finer-grained. For example, PRs that only touch .pre-commit-config.yaml should now not require any runners with GPUs 😁

Removes reliance on pypi.nvidia.com

Contributes to rapidsai/build-planning#241

git grep -i -E 'pypi\.nvidia\.com'
git grep -i -E 'rapids\-init\-pip'

And removed/updated all relevant references. This project does not need any wheels from pypi.nvidia.com at build-time or runtime, it can safely avoid searching that index.

Enforces PEP 639 license metadata in pyproject.toml

Contributes to rapidsai/pre-commit-hooks#95

@jameslamb jameslamb added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Jan 29, 2026
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Jan 29, 2026

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

@github-actions github-actions Bot added the ci label Jan 29, 2026
@jameslamb jameslamb changed the title WIP: expand CI-skipping logic, other small build changes expand CI-skipping logic, other small build changes Jan 30, 2026
@jameslamb jameslamb requested review from csadorf and hcho3 January 30, 2026 03:49
@jameslamb jameslamb marked this pull request as ready for review January 30, 2026 03:49
@jameslamb jameslamb requested review from a team as code owners January 30, 2026 03:49
@jameslamb jameslamb requested a review from msarahan January 30, 2026 03:49
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 30, 2026

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Reorganized CI/CD workflow test groups and dependencies for improved pipeline organization.
    • Updated pre-commit hook version to v1.3.3 and added project license verification checks.
    • Enhanced wheel building process configuration to optimize package dependency resolution.
    • Updated pip initialization configuration across build and test scripts for consistent environment setup.

✏️ Tip: You can customize this high-level summary in your review settings.

Walkthrough

The pull request reorganizes CI workflow test groups in pr.yaml, introduces new conda and wheels test workflows, updates pre-commit configuration with a version bump and new hook, and modifies build/test scripts to set RAPIDS_INIT_PIP_REMOVE_NVIDIA_INDEX to "true". NVIDIA pip index entries are removed from dependencies.yaml.

Changes

Cohort / File(s) Summary
Workflow Configuration
.github/workflows/pr.yaml
Reorganized test file inclusion/exclusion patterns; introduced new test groups test_python_conda and test_python_wheels; updated workflow dependencies to reference new conditional outputs; adjusted conditional execution checks for conda-python-tests and wheels-test triggers.
Build and Test Scripts
ci/build_wheel.sh, ci/build_wheel_cuforest.sh, ci/build_wheel_libcuforest.sh, ci/test_wheel.sh
Added environment variable RAPIDS_INIT_PIP_REMOVE_NVIDIA_INDEX="true" and exported it before rapids-init-pip initialization in each script, affecting pip index handling during wheel building and testing.
Pre-commit Configuration
.pre-commit-config.yaml
Bumped pre-commit hook version from v1.2.1 to v1.3.3; added new verify-pyproject-license hook entry under verify-alpha-spec group with exclusion pattern for top-level pyproject.toml.
Dependencies Configuration
dependencies.yaml
Removed global pip index entries for pypi.nvidia.com across multiple dependency blocks while preserving rapidsai-wheels-nightly index entries.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes


Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

@bdice
Copy link
Copy Markdown
Contributor

bdice commented Jan 30, 2026

/merge

@rapids-bot rapids-bot Bot merged commit 4ae6404 into rapidsai:main Jan 30, 2026
71 checks passed
@hcho3 hcho3 deleted the misc-build-updates branch February 2, 2026 03:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci 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