Add code quality report generation to build pipeline#77
Conversation
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Malcolmnixon
left a comment
There was a problem hiding this comment.
Please run the quality checks - issues are still being introduced - for example docs/quality/introduction.md:17:12 Unknown word (hotspots)
docs/quality/introduction.md
Outdated
|
|
||
| ## Purpose | ||
|
|
||
| This report provides a comprehensive analysis of the TestResults codebase using SonarCloud, |
There was a problem hiding this comment.
This quality report will end up growing to contain more than just SonarCloud analysis - don't try to describe the tool, just describe that the purpose of the document is to act as evidence of good code quality.
There was a problem hiding this comment.
Updated Purpose section to focus on the document serving as evidence of good quality standards rather than describing SonarCloud specifically. (d7efc48)
…urpose section Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Added "hotspots" and "sonarmark" to .cspell.json dictionary. All spell checks now pass. (d7efc48) |
Pull Request
Description
Integrates SonarMark 1.0.0 into the build pipeline to generate a "TestResults Code Quality.pdf" report from SonarCloud analysis results, published alongside existing documentation artifacts.
Changes:
.config/dotnet-tools.json): Addeddemaconsulting.sonarmark1.0.0.cspell.json): Added "hotspots" and "sonarmark" to dictionarydocs/quality/): Created title metadata, introduction, and Pandoc definition matching existing doc structure.github/workflows/build.yaml): Added three steps inbuild-docsjob:Report generation follows the same pattern as requirements and trace matrix documents, using the existing template and tools.
Type of Change
Related Issues
Pre-Submission Checklist
Before submitting this pull request, ensure you have completed the following:
Build and Test
dotnet build --configuration Releasedotnet test --configuration ReleaseCode Quality
dotnet format --verify-no-changesQuality Checks
Please run the following checks before submitting:
cspell "**/*.{md,cs}"markdownlint "**/*.md"yamllint '**/*.{yml,yaml}'Testing
Documentation
Additional Notes
The SonarMark tool queries the existing SonarCloud analysis already performed during the build. No additional analysis overhead is introduced. The SONAR_TOKEN secret is already configured and reused from the existing SonarScanner integration.
The code quality report introduction has been designed to be tool-agnostic, serving as evidence of code quality standards rather than focusing on specific analysis tools, allowing for future expansion beyond SonarCloud.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.