Skip to content

[rush] Fix manual ESRP publish pipeline runs - #5961

Merged
Sean Larkin (TheLarkInn) merged 1 commit into
mainfrom
thelarkinn-esrp-pipeline-parameters
Aug 24, 2026
Merged

[rush] Fix manual ESRP publish pipeline runs#5961
Sean Larkin (TheLarkInn) merged 1 commit into
mainfrom
thelarkinn-esrp-pipeline-parameters

Conversation

@TheLarkInn

Copy link
Copy Markdown
Member

Summary

Azure DevOps disabled manual runs for the Rush and Rush Stack ESRP publish pipelines because their string parameter default was empty and therefore treated as required. Default the commit SHA override to the queued build's source version so both pipelines can be run without supplying a testing override.

Details

Set CommitShaOverride to $(Build.SourceVersion) in both ESRP publish pipeline entry points. Callers can still replace it with another SHA for testing, while normal automatic and manual runs use the source commit selected when queuing the build.

This does not change published package behavior or compatibility.

How it was tested

Validated the final YAML diff and confirmed both pipeline definitions now provide a non-empty default while retaining the existing override flow.

Impacted documentation

N/A

Use the queued build source version as the non-empty default for the optional commit SHA override, allowing Azure DevOps to enable manual runs for both publish pipelines.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 4b8af35a-a354-47a4-803f-f77c9e057d67
Comment thread common/config/azure-pipelines/esrp-publish-rush.yaml
@TheLarkInn
Sean Larkin (TheLarkInn) merged commit e61404a into main Aug 24, 2026
10 checks passed
@TheLarkInn
Sean Larkin (TheLarkInn) deleted the thelarkinn-esrp-pipeline-parameters branch August 24, 2026 18:30
@github-project-automation github-project-automation Bot moved this from Needs triage to Closed in Bug Triage Aug 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Closed

Development

Successfully merging this pull request may close these issues.

2 participants