-
Notifications
You must be signed in to change notification settings - Fork 423
Enable the upload step for the release branches in nightly builds #953
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable the upload step for the release branches in nightly builds #953
Conversation
Signed-off-by: David Gardner <[email protected]>
WalkthroughUpdated GitLab CI to run the Changes
Sequence Diagram(s)sequenceDiagram
autonumber
actor Dev as Developer
participant GitLab as GitLab CI
participant Job as upload:artifactory
participant Script as build_wheel.sh
Dev->>GitLab: Push commit
GitLab->>GitLab: Evaluate job rules
alt Branch matches /^release\/.*$/
GitLab->>Job: Schedule upload:artifactory
Job->>Script: Invoke wheel upload steps
Script-->>Job: Upload actions complete
else Other existing rule matched
GitLab->>Job: Schedule upload:artifactory (existing condition)
Job->>Script: Invoke wheel upload steps
Script-->>Job: Upload actions complete
else No rule matched
GitLab--x Job: Not scheduled
end
Note over GitLab,Job: New regex rule enables job on release/* branches
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🧰 Additional context used📓 Path-based instructions (2){scripts/**,ci/scripts/**}📄 CodeRabbit inference engine (.cursor/rules/general.mdc)
Files:
**/*⚙️ CodeRabbit configuration file
Files:
🔇 Additional comments (1)
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. Comment |
Signed-off-by: David Gardner <[email protected]>
|
/merge |
By Submitting this PR I confirm:
Summary by CodeRabbit