-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Sync eng/common directory with azure-sdk-tools for PR 13750 #47838
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
base: main
Are you sure you want to change the base?
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 syncs the eng/common directory from azure-sdk-tools PR #13750, refactoring the release plan completion marking process to use the azsdk CLI tool instead of the previous DevOps-WorkItem-Helpers approach.
Changes:
- Replaces PowerShell DevOps workitem API calls with the
azsdkCLI tool for marking release plan completion - Adds a new template for installing the azsdk CLI tool in pipeline steps
- Simplifies the release marking logic by delegating workitem management to the
azsdktool
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| eng/common/scripts/Mark-ReleasePlanCompletion.ps1 | Refactored to accept azsdk executable path parameter, removed DevOps-WorkItem-Helpers dependency, simplified logic to use azsdk CLI for release status updates |
| eng/common/pipelines/templates/steps/mark-release-completion.yml | Updated to install azsdk CLI and pass its path to the PowerShell script |
| eng/common/pipelines/templates/steps/install-azsdk-cli.yml | New template that installs the Azure SDK Tools CLI using azure-sdk-mcp.ps1 |
eng/common/pipelines/templates/steps/mark-release-completion.yml
Outdated
Show resolved
Hide resolved
00eee22 to
ebe4c89
Compare
ebe4c89 to
ea830ca
Compare
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Ben Broderick Phillips <ben@benbp.net>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
ea830ca to
7bdc037
Compare
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#13750 See eng/common workflow