Skip to content

Fix ESRP publish stage condition to use stageDependencies expression - #5967

Merged
Sean Larkin (TheLarkInn) merged 1 commit into
mainfrom
copilot/fix-condition-evaluation
Sep 7, 2026
Merged

Fix ESRP publish stage condition to use stageDependencies expression#5967
Sean Larkin (TheLarkInn) merged 1 commit into
mainfrom
copilot/fix-condition-evaluation

Conversation

Copilot AI commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

The Publish stage's condition referenced job outputs using the dependencies. syntax, which is only valid at the job level. At the stage level, this expression silently evaluates to false, causing the stage to be skipped even when the version bump merge check should pass.

Fix

  • Changed the condition on the Publish${{ parameters.StageNameSuffix }} stage to use stageDependencies.Prepare${{ parameters.StageNameSuffix }}.PrepareNpmPublish.outputs['FindBumpRun.IsVersionBumpMerge'], matching the syntax already used for the BumpPipelineRunId/BumpPipelineDefinitionId variables defined later in the same job.
# before
condition: and(succeeded('Prepare${{ parameters.StageNameSuffix }}'), eq(dependencies.Prepare${{ parameters.StageNameSuffix }}.outputs['PrepareNpmPublish.FindBumpRun.IsVersionBumpMerge'], 'true'))

# after
condition: and(succeeded('Prepare${{ parameters.StageNameSuffix }}'), eq(stageDependencies.Prepare${{ parameters.StageNameSuffix }}.PrepareNpmPublish.outputs['FindBumpRun.IsVersionBumpMerge'], 'true'))

Co-authored-by: dmichon-msft <26827560+dmichon-msft@users.noreply.github.com>
@TheLarkInn
Sean Larkin (TheLarkInn) merged commit 77abfe2 into main Sep 7, 2026
10 checks passed
@TheLarkInn
Sean Larkin (TheLarkInn) deleted the copilot/fix-condition-evaluation branch September 7, 2026 12:42
@github-project-automation github-project-automation Bot moved this from Needs triage to Closed in Bug Triage Sep 7, 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.

3 participants