Skip to content

Conversation

@premun
Copy link
Member

@premun premun commented Dec 12, 2025

darc is no longer 6.0 and had a pin so that we don't need to install .NET 8.0 but the pin was removed
dotnet/arcade-services#5443

I pulled these files from the latest VMR build's eng/common

`darc` is no longer 6.0 and had a pin so that we don't need to install .NET 8.0 but the pin was removed
dotnet/arcade-services#5443
Copilot AI review requested due to automatic review settings December 12, 2025 13:56
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds .NET 8.0 SDK installation to support darc tool execution in CI/CD pipelines. According to the description, darc is no longer targeting .NET 6.0 and requires .NET 8.0 to be installed. The PR also includes Azure DevOps pipeline image updates and minor formatting improvements.

Key changes:

  • Adds UseDotNet@2 task to install .NET 8.0.x before running darc operations
  • Updates Windows build agent images from VS2022 to VS2026 preview
  • Adds quotes around pipeline variables and removes trailing whitespace

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
eng/common/core-templates/post-build/post-build.yml Adds .NET 8.0 installation task, updates agent images to VS2026 preview, adds quotes to variable references, and repositions comment
eng/common/core-templates/job/publish-build-assets.yml Adds .NET 8.0 installation task, conditionalizes legacyCredential parameter for DevDiv projects, adds quotes to variable references, removes trailing whitespace

@premun premun enabled auto-merge (squash) December 12, 2025 14:12
@premun premun merged commit 938b677 into dotnet:main Dec 12, 2025
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants