-
Notifications
You must be signed in to change notification settings - Fork 5.1k
Sync eng/common directory with azure-sdk-tools for PR 12478 #53271
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
Sync eng/common directory with azure-sdk-tools for PR 12478 #53271
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
Purpose: Consolidate package validation and DevOps work item update logic into functions, reduce repeated az/auth checks, and add support for supplying explicit PackageInfo files in pipelines.
- Inlines previous Validate-Package and Update-DevOps-Release-WorkItem scripts into function calls.
- Adds new parameters (PackageInfoFiles) and conditional processing logic for package info sources.
- Introduces helper functions (login/access checks, work item update) and updates pipeline template to use them.
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
File | Description |
---|---|
eng/common/scripts/Validate-All-Packages.ps1 | Refactors validation flow into functions; adds PackageInfoFiles handling and consolidated DevOps work item updates. |
eng/common/scripts/Prepare-Release.ps1 | Switches from external script invocation to new Update-DevOpsReleaseWorkItem function. |
eng/common/scripts/Helpers/DevOps-WorkItem-Helpers.ps1 | Adds shared login/access guard functions and encapsulates release work item update logic. |
eng/common/pipelines/templates/steps/validate-all-packages.yml | Updates step parameters to support both artifact list and direct PackageInfoFiles input. |
4023894
to
40ec2cb
Compare
40ec2cb
to
b97f4a7
Compare
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#12478 See eng/common workflow