Disable legacy package-reference pipelines on main - #4647
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
Both pipeline headers still describe triggers that this change disables.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Limits legacy package-reference pipelines while preserving release and scheduled validation.
Changes:
- Restricts PR validation to
release/*. - Removes GitHub
mainCI push triggers while retaining schedules and ADO triggers.
File summaries
| File | Description |
|---|---|
eng/pipelines/sqlclient-pr-package-ref-pipeline.yml |
Limits PR triggers to release branches. |
eng/pipelines/dotnet-sqlclient-ci-package-reference-pipeline.yml |
Disables GitHub main push triggers. |
Review details
- Files reviewed: 2/2 changed files
- Comments generated: 2
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| # GitHub main and release branches. | ||
| - main | ||
| # | ||
| # GOTCHA: Currently disabled on main due to limited resources. |
| - feat/* | ||
| - main | ||
| # | ||
| # GOTCHA: Currently disabled on all but release/* due to limited resources. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4647 +/- ##
==========================================
- Coverage 65.92% 64.48% -1.44%
==========================================
Files 290 284 -6
Lines 44987 67980 +22993
==========================================
+ Hits 29656 43837 +14181
- Misses 15331 24143 +8812
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Description
Limit the legacy package-reference pipelines so they no longer run:
main,dev/*, orfeat/*.main.Package-reference validation remains enabled for release branches. The existing scheduled runs on GitHub
mainand ADOinternal/mainremain enabled, as do CI push triggers for ADOinternal/mainand release branches.This is an engineering-process-only change with no product API or runtime behavior changes.