Skip to content

Apply TemplateDotNetTool changes from PRs #53, #54, #55#107

Merged
Malcolmnixon merged 2 commits intomainfrom
copilot/update-template-repo-changes
Mar 7, 2026
Merged

Apply TemplateDotNetTool changes from PRs #53, #54, #55#107
Malcolmnixon merged 2 commits intomainfrom
copilot/update-template-repo-changes

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 6, 2026

Syncs BuildMark with the latest TemplateDotNetTool template changes from PRs #53, #54, and #55.

Changes

PR #53 — Continuous Compliance documentation

  • README.md: Added Linting Enforcement, Continuous Compliance (with link), SonarCloud Integration, and Requirements Traceability feature bullets; updated Multi-Platform bullet to explicitly include macOS
  • docs/guide/guide.md: Added Continuous Compliance section covering requirements traceability, linting enforcement, automated audit docs, and CodeQL/SonarCloud practices

PR #54 — macOS support, question issue template, agent improvements

  • .github/ISSUE_TEMPLATE/question.yml: New question/help issue template adapted for BuildMark
  • .github/workflows/build.yaml: Added macos-latest to build and integration-test OS matrices; fixed OS_SHORT sed expression from s/windows-latest/win/;s/ubuntu-latest/ubuntu/ to s/-latest// to also handle macOS
  • AGENTS.md: Added Agent Selection Guide (which agent to use for which task); added Test Source Filters section documenting windows@, ubuntu@, macos@, net*.0@, dotnet*.x@ filter prefixes; updated CI/CD section to include macOS
  • requirements.yaml: Added BuildMark-Plt-MacOS requirement with macos@ filtered test links

PR #55 — CodeQL build-mode: manual

  • .github/workflows/build.yaml: Added build-mode: manual to CodeQL init step

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

Related Issues

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 tests pass: dotnet test --configuration Release
  • Self-validation tests pass:
    dotnet run --project src/DemaConsulting.BuildMark --configuration Release --framework net10.0
    --no-build -- --validate
  • 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 '**/*.{yml,yaml}'

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

macOS CI coverage is gated on macos@ test result filters in requirements.yaml — the new BuildMark-Plt-MacOS requirement will only be satisfied once CI runs produce macOS test artifacts.


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

…ompliance, macOS support, CodeQL build-mode

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copilot AI changed the title [WIP] Update with recent changes from template repository Apply TemplateDotNetTool changes from PRs #53, #54, #55 Mar 7, 2026
@Malcolmnixon Malcolmnixon marked this pull request as ready for review March 7, 2026 19:30
Copilot AI review requested due to automatic review settings March 7, 2026 19:30
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

Syncs BuildMark with recent TemplateDotNetTool template updates by expanding CI support to macOS, adding a question/help issue form, and documenting “Continuous Compliance” practices and requirements traceability.

Changes:

  • Add macOS coverage to build and integration-test GitHub Actions matrices, and set CodeQL build-mode: manual.
  • Introduce a “Question / Help” GitHub issue form template.
  • Update documentation and requirements to include Continuous Compliance guidance and a macOS platform requirement with filtered test evidence.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
requirements.yaml Adds a macOS platform requirement (BuildMark-Plt-MacOS) with macos@-filtered test evidence links.
docs/guide/guide.md Documents Continuous Compliance practices and adds a reference link to the methodology.
README.md Updates feature bullets to explicitly include macOS and adds compliance/quality/traceability highlights.
AGENTS.md Adds agent selection guidance and documents test source filter prefixes including macos@.
.github/workflows/build.yaml Expands OS matrices to include macOS; updates OS short-name derivation; configures CodeQL manual build mode.
.github/ISSUE_TEMPLATE/question.yml Adds a question/help issue form aligned with existing bug/feature templates.

💡 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.

@Malcolmnixon Malcolmnixon merged commit e3e351b into main Mar 7, 2026
19 checks passed
@Malcolmnixon Malcolmnixon deleted the copilot/update-template-repo-changes branch March 7, 2026 19:39
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