Skip to content

Sync with TemplateDotNetLibrary: action versions, NuGet bumps, and csproj structure#110

Merged
Malcolmnixon merged 4 commits intomainfrom
copilot/check-for-new-updates
Feb 28, 2026
Merged

Sync with TemplateDotNetLibrary: action versions, NuGet bumps, and csproj structure#110
Malcolmnixon merged 4 commits intomainfrom
copilot/check-for-new-updates

Conversation

Copy link
Contributor

Copilot AI commented Feb 28, 2026

Brings the repository in sync with recent changes merged into the TemplateDotNetLibrary template (PRs #16#19).

Description

GitHub Actions versions

  • upload-artifact@v6@v7 (build.yaml, 6 occurrences)
  • download-artifact@v7@v8 (release.yaml, 2 occurrences)

Dependabot

  • Added groups for github-actions ecosystem — batches all action updates into one PR

NuGet dependency updates

  • SonarAnalyzer.CSharp 10.19.0.13279310.20.0.135146 (both projects)
  • Microsoft.NET.Test.Sdk 18.0.118.3.0 (test project)

csproj structure (template PR #19)

  • LangVersion: 12latest (both projects)
  • Library csproj: split single ItemGroup into labelled Build Tool Dependencies and Code Analysis Dependencies sections with explanatory comments on PrivateAssets/IncludeAssets usage
  • Test csproj: same split into Test Framework Dependencies and Code Analysis Dependencies; added comment on coverlet.collector configuration
  • Windows .NET Framework test target: net462net481

Requirements

  • Updated net462@ source filters in requirements.yaml to net481@ to match the new test target framework

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code quality improvement

Pre-Submission Checklist

Before submitting this pull request, ensure you have completed the following:

Build and Test

  • Code builds successfully: dotnet build --configuration Release
  • All unit tests pass: dotnet test --configuration Release
  • Code produces zero warnings

Code Quality

  • Code formatting is correct: dotnet format --verify-no-changes
  • New code has appropriate XML documentation comments
  • Static analyzer warnings have been addressed

Quality Checks

Please run the following checks before submitting:

  • Spell checker passes: cspell "**/*.{md,cs}"
  • Markdown linter passes: markdownlint "**/*.md"
  • YAML linter passes: yamllint .

Testing

  • Added unit tests for new functionality
  • Updated existing tests if behavior changed
  • All tests follow the AAA (Arrange, Act, Assert) pattern
  • Test coverage is maintained or improved

Documentation

  • Updated README.md (if applicable)
  • Updated ARCHITECTURE.md (if applicable)
  • Added code examples for new features (if applicable)
  • Updated requirements.yaml (if applicable)

Additional Notes

Tracked against template commits: upload-artifact bump, PR #16 (download-artifact v8), PR #17 (dependabot grouping), PR #18 (NuGet bumps), PR #19 (csproj restructure).


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits February 28, 2026 18:39
…ot grouping

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
…to net481

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copilot AI changed the title [WIP] Check for new updates in the system Sync with TemplateDotNetLibrary: action versions, NuGet bumps, and csproj structure Feb 28, 2026
@Malcolmnixon Malcolmnixon marked this pull request as ready for review February 28, 2026 18:56
@Malcolmnixon
Copy link
Contributor

@copilot did you forget to update the test source filters in the requirements file when you updated from net462 to net481?

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 28, 2026

@copilot did you forget to update the test source filters in the requirements file when you updated from net462 to net481?

Good catch — yes, the net462@ source filters in requirements.yaml were missed. Fixed in 3efab47.

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