Support no-op for PR pipeline#60075
Conversation
Add a SkipPrValidation parameter to the .NET pull request pipeline and archetype SDK client template. For manual runs with the flag enabled, emit a no-op stage and skip build/release validation stages. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR adds an opt-in SkipPrValidation boolean parameter to the PR pipeline so that manual runs can complete as a no-op, mirroring the behavior introduced in the Java repo. The flag is threaded from eng/pipelines/pullrequest.yml into the shared stage template, where it conditionally injects a simple “No-op” stage and suppresses build/release validation for manual executions.
Changes:
- Added
SkipPrValidationparameter to the PR pipeline entrypoint (eng/pipelines/pullrequest.yml) and passed it into the archetype stage template. - Updated
archetype-sdk-client.ymlto add a manual-run “NoOp” stage whenSkipPrValidation=true, and to skip Build plus prerelease/release stages in that mode.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| eng/pipelines/templates/stages/archetype-sdk-client.yml | Adds SkipPrValidation parameter, introduces a manual “No-op” stage, and conditions Build/release stages to be skipped when enabled. |
| eng/pipelines/pullrequest.yml | Adds SkipPrValidation pipeline parameter and forwards it to the archetype stage template. |
| - stage: Build | ||
| condition: not(and(eq(${{ parameters.SkipPrValidation }}, true), eq(variables['Build.Reason'], 'Manual'))) | ||
| variables: |
|
@raych1 : Can you please explain the purpose of this feature? I get that this version brings parity with Java, but I'd like to understand the end-to-end scenario and motivation. |
@jsquire : The main motivation is to reduce CI friction for non-SDK changes, especially the common process of syncing skills under the That works well for SDK code changes. However, there are some cases for PRs which might only touch repo automation or other non-SDK related changes - for example, updates under This new feature allows us to trigger the PR validation pipeline with |
Update the PR trigger path exclusions so changes limited to common Azure SDK skill folders do not run PR validation. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Mirrors Azure/azure-sdk-for-java#49558 for the .NET PR pipeline.
SkipPrValidationparameter toeng/pipelines/pullrequest.yml.archetype-sdk-client.yml..github/skills/azsdk-common-*/**from the PR validation trigger so changes limited to common Azure SDK skill folders do not run this pipeline.Validation:
git diff --check