Skip to content

Migrate from PublishBuildArtifacts to PublishPipelineArtifact#16638

Open
mmitche wants to merge 3 commits intodotnet:mainfrom
mmitche:dev/mmitche/publish-pipeline-artifacts
Open

Migrate from PublishBuildArtifacts to PublishPipelineArtifact#16638
mmitche wants to merge 3 commits intodotnet:mainfrom
mmitche:dev/mmitche/publish-pipeline-artifacts

Conversation

@mmitche
Copy link
Copy Markdown
Member

@mmitche mmitche commented Mar 27, 2026

  • Switch publish-build-artifacts to publish-pipeline-artifacts in publish-logs.yml, publish-build-assets.yml, job.yml (both templates and templates-official), and xcopy-msbuild pipeline
  • Split artifacts publish into succeeded() (clean name) and not(succeeded()) (with _Attempt suffix) for retry disambiguation
  • Append _Attempt suffix after coalesce so it applies to both user-provided and default artifact names

To double check:

- Switch publish-build-artifacts to publish-pipeline-artifacts in
  publish-logs.yml, publish-build-assets.yml, job.yml (both templates
  and templates-official), and xcopy-msbuild pipeline
- Split artifacts publish into succeeded() (clean name) and
  not(succeeded()) (with _Attempt suffix) for retry disambiguation
- Append _Attempt suffix after coalesce so it applies to both
  user-provided and default artifact names

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@mmitche mmitche requested a review from akoeplinger March 27, 2026 17:48
@mmitche
Copy link
Copy Markdown
Member Author

mmitche commented Mar 27, 2026

akoeplinger
akoeplinger previously approved these changes Mar 27, 2026
@mmitche
Copy link
Copy Markdown
Member Author

mmitche commented Mar 27, 2026

Needs fix for NuGet validation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants