Split PR devcontainer CI into pip and conda jobs - #22490
Merged
rapids-bot[bot] merged 3 commits intoMay 13, 2026
Merged
Conversation
Use build-in-devcontainer python_package_manager for each variant; require devcontainers-conda in pr-builder and ignore devcontainers-pip for dependency checks.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughSplit the single ChangesDevcontainer Job Separation
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
KyleFromNVIDIA
approved these changes
May 13, 2026
vyasr
approved these changes
May 13, 2026
Contributor
Author
|
/merge |
This was referenced May 13, 2026
3 tasks
rapids-bot Bot
pushed a commit
that referenced
this pull request
May 13, 2026
…22497) Reverts #22490 now that upstream issues breaking pip devcontainers are fixed in cuCascade and rapidsmpf. Authors: - Bradley Dice (https://github.com/bdice) - Kyle Edwards (https://github.com/KyleFromNVIDIA) Approvers: - Kyle Edwards (https://github.com/KyleFromNVIDIA) URL: #22497
3 tasks
This was referenced Jun 5, 2026
3 tasks
This was referenced Jun 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Split the PR workflow devcontainer build into two jobs,
devcontainers-condaanddevcontainers-pip, by passingpython_package_manageras["conda"]or["pip"]tobuild-in-devcontainer.yaml.pr-builderdepends only ondevcontainers-conda;devcontainers-pipis added tochecksignored_pr_jobsso it does not block merge. Sharedpermissionsandbuild_commandare deduplicated with YAML anchors (merge keys are avoided so zizmor can validate the workflow);devcontainers-condais listed first in the workflow file.Checklist