Update build.yaml quality job to use lint scripts

[Copilot]

Pull Request

Description

The quality job in build.yaml was using three individual GitHub Action linters instead of the unified lint.sh/lint.bat scripts, causing drift from the demaconsulting/TemplateDotNetTool template pattern.

Quality job changes:

• Added Setup Node.js (node 24.x) and Setup Python (3.14) steps required by lint.sh to ensure the correct runtimes are available on the CI build agent
• Replaced individual linter actions with a single bash ./lint.sh step:
  • DavidAnson/markdownlint-cli2-action@v22
  • streetsidesoftware/cspell-action@v8
  • ibiqlik/action-yamllint@v3

Lint logic is now centralized in the scripts — future tooling changes only require script updates, not workflow edits. Note that lint.sh also enforces dotnet format --verify-no-changes, expanding the quality gate scope in CI.

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:

• All linters pass: ./lint.sh (Unix/macOS) or cmd /c lint.bat / ./lint.bat (Windows)

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

Aligned with demaconsulting/TemplateDotNetTool template pattern where lint.sh/lint.bat scripts are the single source of truth for quality checks in CI.

[Copilot]

Pull request overview

Updates the reusable GitHub Actions workflow quality job to run the repo’s unified lint scripts instead of separate action-based linters, aligning CI behavior with the lint.sh/lint.bat single-source-of-truth approach.

Changes:

• Added actions/setup-node and actions/setup-python steps to support lint.sh runtime requirements.
• Replaced separate Markdown/spell/YAML lint action steps with a single bash ./lint.sh invocation.

---

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

[Malcolmnixon]

@copilot Your conclusion of no changes needed is utterly incorrect. I can clearly see the quality-checks job in build.yaml of this repository uses DavidAnson/markdownlint-cli2-action@v22, streetsidesoftware/cspell-action@v8, and ibiqlik/action-yamllint@v3 whereas the same file in TemplateDotNetTool (https://github.com/demaconsulting/TemplateDotNetTool/blob/main/.github/workflows/build.yaml) does not use those actions and instead uses bash ./lint.sh

[Copilot]

@copilot Your conclusion of no changes needed is utterly incorrect. I can clearly see the quality-checks job in build.yaml of this repository uses `DavidAnson/markdownlint-cli2-actio...

Apologies for the confusion — I had referenced the wrong template repo. Confirmed that demaconsulting/TemplateDotNetTool uses Setup Node.js (24.x), Setup Python (3.14), and bash ./lint.sh. Restored the quality job to that pattern in 41aed77.

[Malcolmnixon]

@copilot Your conclusion of no changes needed is utterly incorrect. I can clearly see the quality-checks job in build.yaml of this repository uses `DavidAnson/markdownlint-cli2-actio...

Apologies for the confusion — I had referenced the wrong template repo. Confirmed that demaconsulting/TemplateDotNetTool uses Setup Node.js (24.x), Setup Python (3.14), and bash ./lint.sh. Restored the quality job to that pattern in 41aed77.

I think part of that was me copy-pasting the wrong link.