-
Notifications
You must be signed in to change notification settings - Fork 337
Add Regen Preview Pipeline for azure-sdk-for-net preview PRs #9600
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
Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>
Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>
…ttern Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>
| npm install -g @azure-tools/typespec-client-generator-cli@latest | ||
| displayName: Install tsp-client | ||
|
|
||
| - task: PowerShell@2 |
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.
@copilot why do we need this? The regen preview script should already be handling updating the deps for us.
Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>
packages/http-client-csharp/eng/scripts/Submit-RegenPreviewPr.ps1
Outdated
Show resolved
Hide resolved
packages/http-client-csharp/eng/scripts/Submit-RegenPreviewPr.ps1
Outdated
Show resolved
Hide resolved
packages/http-client-csharp/eng/scripts/Submit-RegenPreviewPr.ps1
Outdated
Show resolved
Hide resolved
Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>
|
|
||
| Push-Location $AzureSdkRepoPath | ||
| try { | ||
| # Configure git |
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.
@copilot why is this needed? We are already pulling down the repo. This doesn't seem required for the existing publish pipeline and script?
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.
You're correct. The git config commands are not needed since the pipeline already handles repository initialization with git init and git checkout. Removed the unnecessary git config lines to match the pattern in Submit-AzureSdkForNetPr.ps1. (1668a32)
Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>
Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>
Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>
Implementation Complete ✅
regen-preview.yamlpipeline file inpackages/http-client-csharp/eng/pipeline/Submit-RegenPreviewPr.ps1script followingSubmit-AzureSdkForNetPr.ps1patternSummary
Created a new Azure DevOps pipeline (
regen-preview.yaml) with dedicated PowerShell script (Submit-RegenPreviewPr.ps1) that provides a manual workflow for regenerating libraries in azure-sdk-for-net and creating preview PRs.The pipeline relies on RegenPreview.ps1 to handle all package building and dependency management from source. The generated PR title dynamically shows failure status as a prefix (e.g., "(Failed) Regen Preview: ...") and the PR body displays the specific package names based on the selected library type. Only changes in the
sdk/folder are staged for the preview PR. The script follows the same pattern as Submit-AzureSdkForNetPr.ps1.The pipeline includes a step that safely parses the regen-report.json file and displays detailed information about any failed library regenerations directly in the pipeline output. The step uses
$ErrorActionPreference = 'Continue'and wraps all logic in a try-catch block to ensure parsing errors don't fail the build. If parsing fails, a warning is displayed and users are directed to the artifact. This makes it easier to diagnose issues without downloading the artifact, while ensuring the pipeline remains resilient.Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.