Skip to content

chore: bring in updates from TemplateDotNetLibrary template repo#13

Merged
Malcolmnixon merged 4 commits intomainfrom
copilot/update-from-template-repo
Mar 8, 2026
Merged

chore: bring in updates from TemplateDotNetLibrary template repo#13
Malcolmnixon merged 4 commits intomainfrom
copilot/update-from-template-repo

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 7, 2026

Pull Request

Description

Syncs infrastructure files with the latest TemplateDotNetLibrary template.

.github/workflows/build.yaml

  • Added macOS to build matrix ([windows-latest, ubuntu-latest, macos-latest])
  • Added build-mode: manual to CodeQL init
  • Added OTS self-validation steps for all tools (VersionMark, ReqStream, BuildMark, SarifMark, SonarMark) generating TRX proof files for requirements traceability
  • Unified all job outputs under a single artifacts/ directory; simplified artifact downloads in build-docs job to a single artifacts-* pattern
  • Added structured section comments throughout all jobs

.github/workflows/release.yaml

  • Fixed artifact name: artifacts-ubuntu-latestpackages-ubuntu-latest to align with new build workflow naming

.github/ISSUE_TEMPLATE/question.yml

  • Added missing ❓ Question/Help issue template

requirements.yaml

  • Added macOS platform requirement (CACH-PLT-007) with macos@ test source filter
  • Added macos@ source filter prefix to header comments
  • Added OTS Software section with requirements for MSTest, ReqStream, BuildMark, VersionMark, SarifMark, and SonarMark

AGENTS.md

  • Added Agent Selection Guide mapping common tasks to specialized agents
  • Updated CI/CD description to include macOS
  • Fixed encoding note: UTF-8+BOM → UTF-8
  • Updated reqstream CI enforcement command path to artifacts/**/*.trx

CONTRIBUTING.md

  • Fixed Assert.HasCount argument order to match MSTest v4 API: (expectedCount, collection)
  • Fixed encoding note: UTF-8 with BOM → UTF-8

README.md

  • Updated multi-platform feature to include macOS
  • Updated multi-runtime feature to include .NET Standard 2.0
  • Added new feature bullets: Linting Enforcement, Continuous Compliance, SonarCloud Integration, Requirements Traceability
  • Added [link-continuous-compliance] reference

docs/guide/guide.md

  • Added Continuous Compliance section with methodology link and key practices

.config/dotnet-tools.json

  • Updated demaconsulting.reqstream 1.3.01.4.1

src/DemaConsulting.NuGet.Caching/DemaConsulting.NuGet.Caching.csproj

  • Updated Polyfill 9.13.09.18.0

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

Infrastructure-only changes; no production code or test logic modified.


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

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copilot AI changed the title [WIP] Update project with changes from template repository chore: bring in updates from TemplateDotNetLibrary template repo Mar 7, 2026
@Malcolmnixon
Copy link
Copy Markdown
Member

@copilot Are you sure you found all changes - please check the template repos PRs. For example im not seeing the new macOS requirements which suggests you failed to track down all changes.

…nts, dep updates)

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

Copilot AI commented Mar 7, 2026

@copilot Are you sure you found all changes - please check the template repos PRs. For example im not seeing the new macOS requirements which suggests you failed to track down all ch...

Good catch - I missed several changes from the template PRs. Commit 91bfd76 adds:

@Malcolmnixon Malcolmnixon marked this pull request as ready for review March 8, 2026 00:16
Copilot AI review requested due to automatic review settings March 8, 2026 00:16
Copy link
Copy Markdown
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 syncs this repository’s CI/CD and compliance infrastructure with the latest TemplateDotNetLibrary template, expanding platform coverage (macOS) and strengthening requirements/traceability and reporting outputs.

Changes:

  • Updated GitHub Actions workflows to add macOS builds, unify outputs under artifacts/, and add OTS tool self-validation TRX evidence generation.
  • Extended requirements.yaml with macOS platform requirements and an “OTS Software” section linking tools to evidence.
  • Refreshed documentation and repo guidance (README/guide/agents/contributing) and bumped tool/dependency versions (ReqStream, Polyfill).

Reviewed changes

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

Show a summary per file
File Description
.github/workflows/build.yaml Adds macOS to the build matrix, unifies CI artifacts under artifacts/, and adds OTS tool self-validation + tool version capture.
.github/workflows/release.yaml Aligns downloaded package artifact name with the new build workflow naming.
.github/ISSUE_TEMPLATE/question.yml Adds a Question/Help issue form template.
requirements.yaml Adds macOS platform requirement and an OTS Software section with tool-related requirements tied to TRX evidence.
AGENTS.md Updates repo agent guidance, CI/CD description (macOS), and ReqStream enforcement path to artifacts/**/*.trx.
CONTRIBUTING.md Updates encoding guidance and corrects MSTest v4 Assert.HasCount argument order.
README.md Updates feature list for macOS/.NET Standard 2.0 and adds compliance/traceability/quality feature bullets + link.
docs/guide/guide.md Adds a Continuous Compliance section and link reference.
.config/dotnet-tools.json Bumps demaconsulting.reqstream version.
src/DemaConsulting.NuGet.Caching/DemaConsulting.NuGet.Caching.csproj Bumps Polyfill package version.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Malcolmnixon Malcolmnixon merged commit 22e0eea into main Mar 8, 2026
6 checks passed
@Malcolmnixon Malcolmnixon deleted the copilot/update-from-template-repo branch March 8, 2026 00:58
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