CHORE: Remove dead pipelines: build-whl, dummy-release, official-release#550
Open
bewithgaurav wants to merge 2 commits intomainfrom
Open
CHORE: Remove dead pipelines: build-whl, dummy-release, official-release#550bewithgaurav wants to merge 2 commits intomainfrom
bewithgaurav wants to merge 2 commits intomainfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Removes Azure DevOps pipeline YAML definitions that are believed to be unused, reducing maintenance surface for legacy pipeline configurations.
Changes:
- Deleted the legacy wheel build pipeline definition under
eng/pipelines/. - Deleted the legacy dummy ESRP release pipeline definition under
eng/pipelines/. - Deleted the legacy official ESRP release pipeline definition under
eng/pipelines/.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| eng/pipelines/build-whl-pipeline.yml | Removes the legacy wheel build pipeline YAML. |
| eng/pipelines/dummy-release-pipeline.yml | Removes the legacy dummy ESRP release pipeline YAML. |
| eng/pipelines/official-release-pipeline.yml | Removes the legacy official ESRP release pipeline YAML. |
Comments suppressed due to low confidence (2)
eng/pipelines/build-whl-pipeline.yml:1
- PR description says the unused pipeline files being deleted are under
OneBranchPipelines/(includingbuild-release-package-pipeline.yml), but this PR actually deletes pipeline YAMLs undereng/pipelines/(build-whl-pipeline.yml,dummy-release-pipeline.yml,official-release-pipeline.yml). Please update the PR description (or the file deletions) so the intent and review surface match what’s changing.
eng/pipelines/build-whl-pipeline.yml:1 OneBranchPipelines/stress-test-pipeline.ymlstill downloads artifacts from pipeline definition2162, annotated as the “build-whl-pipeline definition ID”. Deleting this pipeline YAML risks breaking the stress-test pipeline (scheduled) unless that reference is updated to the replacement build pipeline / artifact source.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
bewithgaurav
changed the title
📊 Code Coverage Report
Diff CoverageDiff: main...HEAD, staged and unstaged changesNo lines with coverage information in this diff. 📋 Files Needing Attention📉 Files with overall lowest coverage (click to expand)mssql_python.pybind.logger_bridge.cpp: 59.2%
mssql_python.pybind.ddbc_bindings.h: 67.9%
mssql_python.row.py: 70.5%
mssql_python.pybind.logger_bridge.hpp: 70.8%
mssql_python.pybind.ddbc_bindings.cpp: 74.6%
mssql_python.pybind.connection.connection.cpp: 76.2%
mssql_python.__init__.py: 77.3%
mssql_python.ddbc_bindings.py: 79.6%
mssql_python.pybind.connection.connection_pool.cpp: 79.6%
mssql_python.connection.py: 85.3%🔗 Quick Links
|
saurabh500
approved these changes
Apr 29, 2026
Contributor
saurabh500
left a comment
saurabh500
left a comment
There was a problem hiding this comment.
I love to see dead code deletion ! :)
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.
Work Item / Issue Reference
Summary
Deletes the following unused pipeline files from
OneBranchPipelines/:build-release-package-pipeline.ymldummy-release-pipeline.ymlofficial-release-pipeline.ymlThese pipelines are no longer needed and were dead code.