Repository navigation
Remove constraints argument from rapids-generate-pip-constraints - #261
Conversation
With rapidsai/gha-tools#282 merged, we no longer need the third `constraints` argument. Remove it. Issue: rapidsai/build-planning#311
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Update constraint invocation ci/test_wheel.sh |
The constraint-generation command no longer passes the trailing constraints argument. |
Priority:
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Other
Possibly related PRs
- rapidsai/nvforest#260: This PR adds the same argument to the same command, while this change removes it.
Suggested labels: improvement, non-breaking
Suggested reviewers: chyunsu3
Merge Risk: 
Minimal · up to 41fc2
-
The constraint-generation change preserves the wheel test's existing pip constraint workflow.
Pre-merge checks: 
5
-
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files.
-
Finishing Touches: 💡 1
-
🛠️ Fix failing CI checks: 💡
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
|
/merge |
With rapidsai/gha-tools#282 merged, we no longer
need the third
constraintsargument. Remove it.Issue: rapidsai/build-planning#311