Skip to content

Conversation

@jameslamb
Copy link
Member

@jameslamb jameslamb commented Jan 14, 2026

Contributes to rapidsai/build-planning#236

Updates branch for GitHub Actions workflows from @cuda-13.1.0 to @main now that rapidsai/shared-workflows#483 is merged.

Summary by CodeRabbit

  • Chores
    • Updated GitHub Actions workflow configurations to use main branch tracking instead of version-specific references across build, test, and release pipelines.

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

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

copy-pr-bot bot commented Jan 14, 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 jameslamb marked this pull request as ready for review January 14, 2026 16:11
@jameslamb jameslamb requested a review from a team as a code owner January 14, 2026 16:11
@jameslamb jameslamb requested a review from vyasr January 14, 2026 16:11
@coderabbitai
Copy link

coderabbitai bot commented Jan 14, 2026

📝 Walkthrough

Walkthrough

Updated external shared workflow references across four GitHub Actions workflow files from a specific CUDA 13.1.0 version tag to the main branch. No changes to job logic, control flow, or input parameters.

Changes

Cohort / File(s) Summary
Workflow Version Reference Updates
.github/workflows/build.yaml, .github/workflows/pr.yaml, .github/workflows/test.yaml, .github/workflows/trigger-breaking-change-alert.yaml
Replaced all @cuda-13.1.0 references with @main across reusable workflow invocations in cpp-build, python-build, upload-conda, wheel-build/publish, conda-python-tests, conda-cpp-tests, wheels-test, and breaking-change-alert jobs. No changes to job orchestration, conditional logic, or input configurations.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Use main shared-workflows branch' directly and clearly summarizes the main change across all modified workflow files—updating all references from the cuda-13.1.0 tag to the main branch.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.



📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 72a757a and 0f85d6d.

📒 Files selected for processing (4)
  • .github/workflows/build.yaml
  • .github/workflows/pr.yaml
  • .github/workflows/test.yaml
  • .github/workflows/trigger-breaking-change-alert.yaml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (6)
  • GitHub Check: wheel-build-cuopt-mps-parser / 13.1.0, 3.12, arm64, rockylinux8
  • GitHub Check: wheel-build-cuopt-mps-parser / 13.1.0, 3.10, arm64, rockylinux8
  • GitHub Check: wheel-build-cuopt-mps-parser / 13.1.0, 3.11, arm64, rockylinux8
  • GitHub Check: wheel-build-cuopt-mps-parser / 13.1.0, 3.13, arm64, rockylinux8
  • GitHub Check: wheel-build-cuopt-sh-client / 13.1.0, 3.10, amd64, rockylinux8
  • GitHub Check: checks / check-style
🔇 Additional comments (4)
.github/workflows/test.yaml (1)

30-30: LGTM - workflow references updated to @main.

The shared workflow references are consistently updated. Using @main ensures you'll receive the latest fixes and features from the shared-workflows repository automatically. This aligns with the upstream changes in rapidsai/shared-workflows#483.

Also applies to: 45-45, 61-61, 76-76, 92-92

.github/workflows/trigger-breaking-change-alert.yaml (1)

18-18: LGTM!

The breaking-change-alert workflow reference is updated consistently with the other workflow files.

.github/workflows/build.yaml (1)

47-47: LGTM - all shared workflow references updated.

All 14 external workflow references are consistently updated to @main. The local workflow reference at line 249 (./.github/workflows/build_test_publish_images.yaml) is correctly left unchanged as a relative path.

Also applies to: 57-57, 67-67, 75-75, 90-90, 101-101, 114-114, 125-125, 137-137, 147-147, 162-162, 173-173, 187-187, 203-203

.github/workflows/pr.yaml (1)

37-37: LGTM - all shared workflow references updated.

All 15 external workflow references are consistently updated to @main. The local workflow reference at line 329 (./.github/workflows/self_hosted_service_test.yaml) is correctly left unchanged.

Also applies to: 111-111, 176-176, 182-182, 189-189, 205-205, 212-212, 229-229, 241-241, 253-253, 264-264, 273-273, 289-289, 301-301, 313-313

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@rgsl888prabhu rgsl888prabhu merged commit 56c43d2 into NVIDIA:main Jan 14, 2026
35 checks passed
@jameslamb jameslamb deleted the use-main branch January 14, 2026 16:29
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