Skip to content

Commit c4cbe74

Browse files
akoeplingerCopilotCopilotmmitche
authored
Mark publishing artifacts as non-production and fix boolean property pass-through in 1ES publish template (dotnet#16672)
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: akoeplinger <1376924+akoeplinger@users.noreply.github.com> Co-authored-by: mmitche <8725170+mmitche@users.noreply.github.com>
1 parent 1ad1384 commit c4cbe74

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

eng/common/core-templates/job/publish-build-assets.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -173,6 +173,7 @@ jobs:
173173
artifactName: AssetManifests
174174
displayName: 'Publish Merged Manifest'
175175
retryCountOnTaskFailure: 10 # for any files being locked
176+
isProduction: false # just metadata for publishing
176177

177178
- template: /eng/common/core-templates/steps/publish-pipeline-artifacts.yml
178179
parameters:
@@ -182,6 +183,7 @@ jobs:
182183
targetPath: '$(Build.StagingDirectory)/ReleaseConfigs'
183184
artifactName: ReleaseConfigs
184185
retryCountOnTaskFailure: 10 # for any files being locked
186+
isProduction: false # just metadata for publishing
185187

186188
- ${{ if or(eq(parameters.publishAssetsImmediately, 'true'), eq(parameters.isAssetlessBuild, 'true')) }}:
187189
- template: /eng/common/core-templates/post-build/setup-maestro-vars.yml

eng/common/templates-official/steps/publish-pipeline-artifacts.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ steps:
2424
artifactName: ${{ parameters.args.artifactName }}
2525
${{ if parameters.args.properties }}:
2626
properties: ${{ parameters.args.properties }}
27-
${{ if parameters.args.sbomEnabled }}:
27+
${{ if ne(parameters.args.sbomEnabled, '') }}:
2828
sbomEnabled: ${{ parameters.args.sbomEnabled }}
29-
${{ if parameters.args.isProduction }}:
29+
${{ if ne(parameters.args.isProduction, '') }}:
3030
isProduction: ${{ parameters.args.isProduction }}

0 commit comments

Comments
 (0)