-
Notifications
You must be signed in to change notification settings - Fork 317
Enable new PR pipelines #3759
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 new PR pipelines #3759
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR enables the new PR validation pipelines for all target branches by removing temporary CI debugging restrictions. The pipelines will now trigger PR validation builds for pull requests targeting dev/*, feat/*, and main branches instead of being limited to the dev/paul/ci-debug* test branches.
- Removed temporary branch restrictions from PR pipeline configurations
- Enabled PR validation for production branch patterns (
dev/*,feat/*,main) - Applied changes consistently across both project-reference and package-reference pipelines
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| eng/pipelines/sqlclient-pr-project-ref-pipeline.yml | Updated PR trigger branches from test-only pattern to production patterns |
| eng/pipelines/sqlclient-pr-package-ref-pipeline.yml | Updated PR trigger branches from test-only pattern to production patterns |
- Updated all path specs to include all files/directories that affect builds. - Improved documentation.
- Reduced Debug test job timeout from 180 to 110 minutes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3759 +/- ##
==========================================
+ Coverage 76.47% 76.61% +0.13%
==========================================
Files 273 273
Lines 44180 44180
==========================================
+ Hits 33787 33848 +61
+ Misses 10393 10332 -61
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Description
This PR enables the new PR pipelines for all desired branches.
I still need to figure out how to configure GitHub to require these PR pipelines to pass as branch checks. They don't currently appear in the GitHub UI as checks I can select.