Skip to content

Bring in recent changes from the template repo#31

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/bring-in-recent-changes
Draft

Bring in recent changes from the template repo#31
Copilot wants to merge 2 commits intomainfrom
copilot/bring-in-recent-changes

Conversation

Copy link
Contributor

Copilot AI commented Mar 6, 2026

Syncs VersionMark with recent improvements from the TemplateDotNetTool template (PRs #41–#55).

Description

CI/CD Workflow (build.yaml)

  • quality-checks job: Add dotnet setup, tool restore, dotnet versionmark capture + self-validation, artifact upload, and section guidance comments
  • build job: Add macos-latest to strategy matrix; insert version capture and self-validation steps between Restore Tools and Restore Dependencies; add section comments
  • codeql job: Add build-mode: manual to codeql-action/init@v4
  • integration-test job: Add macos-latest to strategy matrix; add Restore Tools step; normalize OS_SHORT with sed 's/-latest//' (consistent across all platforms); add versionmark to captured tools
  • build-docs job: Move Setup Node.js into INSTALL DEPENDENCIES section (after artifact downloads); add versionmark to tool capture list
  • All jobs: Add # Downstream projects: guidance comments to each # === section

Tool Infrastructure

  • .config/dotnet-tools.json: Add demaconsulting.versionmark 1.0.0 — enables dotnet versionmark in quality-checks and build jobs
  • .versionmark.yaml: Add versionmark tool entry (reads from dotnet tool list)

Documentation & Agent Guidance

  • AGENTS.md: Add Agent Selection Guide (task → agent mapping); add macos@TestName test source filter; update CI/CD section to reflect macOS support
  • README.md: Add Continuous Compliance feature entry with link to the methodology
  • docs/guide/guide.md: Add Continuous Compliance section after Introduction
  • test/.csproj: Add <!-- Build Configuration --> and <!-- Code Quality Configuration --> section comments; match template property order
  • .gitignore: Add # VersionMark captures (generated during CI/CD) heading; fix npm-debug.log ordering

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
  • Self-validation tests pass:
    dotnet run --project src/DemaConsulting.VersionMark --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 .

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

The quality-checks and build jobs now use dotnet versionmark (from .config/dotnet-tools.json, pointing to the released 1.0.0) for capture and self-validation, while integration-test and build-docs continue to use the globally installed versionmark built from the current PR — preserving the intended self-test behavior.


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copilot AI changed the title [WIP] Update with recent changes from template repository Bring in recent changes from the template repo Mar 7, 2026
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