Skip to content

Merge latest Library.Template#689

Merged
AArnott merged 18 commits intomainfrom
dev/andarno/libtemplateUpdate
Mar 6, 2026
Merged

Merge latest Library.Template#689
AArnott merged 18 commits intomainfrom
dev/andarno/libtemplateUpdate

Conversation

@AArnott
Copy link
Copy Markdown
Member

@AArnott AArnott commented Mar 6, 2026

  • Only disable NBGV_GitEngine when necessary
  • Update dotnet monorepo to 4.14.0
  • Merge pull request 481 from AArnott/renovate/docfx-2.x
  • Update mcr.microsoft.com/dotnet/sdk:10.0.103 Docker digest to e362a8d (480)
  • Update dependency dotnet-coverage to v18.5.1 (483)
  • Update dependency Microsoft.Testing.Extensions.CodeCoverage to 18.5.1 (484)
  • Update GitHub Artifact Actions (482)
  • Bump DotNetRepoTools to 1.2.1
  • Fix package downgrade breaks after Roslyn 4.14 bump
  • Update dotnet monorepo to v5
  • Update dependency Microsoft.Testing.Extensions.CodeCoverage to 18.5.2 (486)
  • Update dependency dotnet-coverage to v18.5.2 (485)

AArnott and others added 18 commits February 23, 2026 12:55
… (480)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
… (484)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 6, 2026 16:18
Copy link
Copy Markdown

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

Updates repository template/tooling dependencies and build guidance, aligning the repo with the latest Library.Template and newer CI/toolchain components.

Changes:

  • Bump Roslyn/CodeAnalysis and Microsoft.Testing.Platform-related package/tool versions.
  • Update GitHub Actions artifact publishing to actions/upload-artifact@v7 (pinned SHA).
  • Replace blanket “disable NBGV” guidance with a shallow-clone priming script and updated Copilot instructions.

Reviewed changes

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

Show a summary per file
File Description
Directory.Packages.props Updates centrally managed versions (notably CodeAnalysisVersion and CodeCoverage extension).
.github/copilot-instructions.md Updates build/test prep guidance to use the new priming script.
.github/actions/publish-artifacts/action.yaml Updates actions/upload-artifact pinned SHA to v7.0.0 across artifact uploads.
.github/Prime-ForCopilot.ps1 Adds a helper script to disable NBGV Git engine for shallow clones.
.config/dotnet-tools.json Updates dotnet tool versions (dotnet-coverage, docfx, DotNetRepoTools).
Comments suppressed due to low confidence (1)

Directory.Packages.props:16

  • CodeAnalysisVersion was bumped to 5.0.0, but Microsoft.CodeAnalysis.Analyzers remains pinned to 4.14.0 in the same file. This mixed Roslyn major version set can lead to restore/build issues or analyzer/SDK incompatibilities. Consider updating the analyzer package to a matching version (or documenting/justifying why it must remain at 4.14.0).
    <CodeAnalysisVersion>5.0.0</CodeAnalysisVersion>
    <CodefixTestingVersion>1.1.2</CodefixTestingVersion>
  </PropertyGroup>
  <ItemGroup>
    <PackageVersion Include="DiffPlex" Version="1.9.0" />
    <PackageVersion Include="Microsoft.CodeAnalysis.Analyzers" Version="4.14.0" />
    <PackageVersion Include="Microsoft.CodeAnalysis.Common" version="$(CodeAnalysisVersion)" />

You can also share your feedback on Copilot code review. Take the survey.

Comment thread .github/copilot-instructions.md
Comment thread .github/Prime-ForCopilot.ps1
@AArnott AArnott enabled auto-merge March 6, 2026 17:16
@AArnott AArnott merged commit 71f471d into main Mar 6, 2026
13 checks passed
@AArnott AArnott deleted the dev/andarno/libtemplateUpdate branch March 6, 2026 18:37
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.

3 participants