Skip to content

Commit 30bd106

Browse files
RyAuldCopilot
andcommitted
Fix: allow Build/Publish stages to proceed on SucceededWithIssues
The cryptography ceiling check runs with continueOnError: true, which causes the CI stage to finish as SucceededWithIssues rather than Succeeded. The strict eq(..., 'Succeeded') conditions on Build, PublishMSALPython, and PublishPyPI stages caused them to skip. Change all three conditions to use in(..., 'Succeeded', 'SucceededWithIssues') so warning-only tasks don't block the release. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent cfbd078 commit 30bd106

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

.Pipelines/pipeline-publish.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ stages:
4444
- stage: Build
4545
displayName: 'Build package'
4646
dependsOn: CI
47-
condition: eq(dependencies.CI.result, 'Succeeded')
47+
condition: in(dependencies.CI.result, 'Succeeded', 'SucceededWithIssues')
4848
jobs:
4949
- job: BuildDist
5050
displayName: 'Build sdist + wheel (Python 3.12)'
@@ -83,7 +83,7 @@ stages:
8383
dependsOn: Build
8484
condition: >
8585
and(
86-
eq(dependencies.Build.result, 'Succeeded'),
86+
in(dependencies.Build.result, 'Succeeded', 'SucceededWithIssues'),
8787
eq('${{ parameters.publishTarget }}', 'test.pypi.org (Preview / RC)')
8888
)
8989
jobs:
@@ -140,7 +140,7 @@ stages:
140140
dependsOn: Build
141141
condition: >
142142
and(
143-
eq(dependencies.Build.result, 'Succeeded'),
143+
in(dependencies.Build.result, 'Succeeded', 'SucceededWithIssues'),
144144
eq('${{ parameters.publishTarget }}', 'pypi.org (ESRP Production)')
145145
)
146146
jobs:

0 commit comments

Comments
 (0)