Skip to content

Upgrade to V4 Arcade publishing#565

Open
mmitche wants to merge 4 commits into
dotnet:mainfrom
mmitche:dev/mmitche/arcade-v4-publishing
Open

Upgrade to V4 Arcade publishing#565
mmitche wants to merge 4 commits into
dotnet:mainfrom
mmitche:dev/mmitche/arcade-v4-publishing

Conversation

@mmitche

@mmitche mmitche commented Jul 15, 2026

Copy link
Copy Markdown
Member

Summary

  • Upgrade deployment-tools to Arcade V4 publishing.
  • Set PublishingVersion to 4 and wire official publish-build-assets/post-build templates for V4.
  • Route Windows build jobs through /eng/common/templates-official/job/job.yml@self so Arcade's wrapper emits the V4 $(System.PhaseName)_Artifacts output.
  • Retain minimal staging of packages/manifests into $(Build.ArtifactStagingDirectory)/artifacts because this repo's build writes them under repo artifacts/ rather than directly into the wrapper staging directory.
  • Update System.Security.Cryptography.Xml to 8.0.4 to unblock current official NuGet audit errors.

Validation tracking

@mmitche
mmitche requested a review from a team as a code owner July 15, 2026 03:40
@mmitche
mmitche force-pushed the dev/mmitche/arcade-v4-publishing branch from 2b185e7 to 57d3973 Compare July 15, 2026 16:45
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@mmitche
mmitche force-pushed the dev/mmitche/arcade-v4-publishing branch from 57d3973 to 87af2c9 Compare July 15, 2026 17:21
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: cdd830ac-44ac-4fe8-96e7-44dfffb28763
Comment thread eng/jobs/windows-build.yml Outdated
displayName: Setup Private Feeds Credentials
condition: eq(variables['Agent.OS'], 'Windows_NT')
inputs:
filePath: $(Build.SourcesDirectory)/eng/common/SetupNugetSources.ps1

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replace this with the enable-internal-sources yml template.

Comment thread eng/jobs/windows-build.yml Outdated
displayName: 'Push NetCore.Launcher NuPkgs to DevDiv VS feed'
condition: succeeded()
- ${{ if and(ne(variables['System.TeamProject'], 'public'), notin(variables['Build.Reason'], 'PullRequest')) }}:
- powershell: |

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still think this is incorrect. This seems to imply that publishing is not beign used properly in the repository. When publishing is enabled and items are in the publishing itemgroups, then they will get copied to the staging directories. I think all of these steps should be eliminated.

mmitche and others added 2 commits July 17, 2026 12:47
…s automatically

Arcade's job.yml template with publishingVersion:4 + enablePublishing runs
Publish.proj, which locally stages shipping/nonshipping packages, asset
manifests, and PDBs to $(Build.ArtifactStagingDirectory)/artifacts and
publishes them as $(System.PhaseName)_Artifacts. The manual staging steps and
the IntermediateUnsignedArtifacts/PdbArtifacts outputs duplicated this and are
removed.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: cdd830ac-44ac-4fe8-96e7-44dfffb28763
Replace the manual SetupNugetSources.ps1 + PAT step with the standard
enable-internal-sources.yml template, which uses federated auth
(dnceng-artifacts-feeds-read) and handles NuGetAuthenticate.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: cdd830ac-44ac-4fe8-96e7-44dfffb28763
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.

1 participant