Tags: demaconsulting/ReviewMark
Tags
Fix Sonar code quality issues (#61) * Initial plan * Fix Sonar code quality issues (assertions, complexity, constants) Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/f945d2b8-41af-4649-a4d9-8f2f9931faf4 Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * Update design doc to document ValidateEvidenceSource and ValidateReviews helpers Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/befcd8fc-6785-4437-ac84-b24b608be0df Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * Make ValidateEvidenceSource/ValidateReviews private; fix design doc ownership and remove internal claim Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/b5ac33dd-251a-49c3-a3d0-b3e4ee7e7ff5 Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * Fix MD013 line-length lint issues in review-mark-configuration.md Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/765672bf-58eb-4dd6-a0d2-c6b72fa16651 Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * Restore ValidateEvidenceSource and ValidateReviews; build clean, lint passes Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/4bc273ca-a180-43f3-be79-f5d423322c23 Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * Fix duplicate-ID error message to match test expectations; build clean, 220/220 tests pass, lint clean Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/f2e8200d-b38a-4780-bb56-320d96c3e1cb Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Bump the nuget-dependencies group with 7 updates (#52) Bumps demaconsulting.buildmark from 0.5.0 to 0.6.0 Bumps Microsoft.NET.Test.Sdk from 18.3.0 to 18.4.0 Bumps MSTest.TestAdapter from 4.1.0 to 4.2.1 Bumps MSTest.TestFramework from 4.1.0 to 4.2.1 Bumps Polyfill from 10.0.0 to 10.3.0 Bumps SonarAnalyzer.CSharp from 10.22.0.136894 to 10.23.0.137933 Bumps YamlDotNet from 16.3.0 to 17.0.1 --- updated-dependencies: - dependency-name: demaconsulting.buildmark dependency-version: 0.6.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-dependencies - dependency-name: Microsoft.NET.Test.Sdk dependency-version: 18.4.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-dependencies - dependency-name: MSTest.TestAdapter dependency-version: 4.2.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-dependencies - dependency-name: MSTest.TestFramework dependency-version: 4.2.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-dependencies - dependency-name: Polyfill dependency-version: 10.3.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-dependencies - dependency-name: SonarAnalyzer.CSharp dependency-version: 10.23.0.137933 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-dependencies - dependency-name: SonarAnalyzer.CSharp dependency-version: 10.23.0.137933 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-dependencies - dependency-name: YamlDotNet dependency-version: 17.0.1 dependency-type: direct:production update-type: version-update:semver-major dependency-group: nuget-dependencies ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
fix: formal review fixes — results file directory creation and requir… …ements scope cleanup (#47) * fix: remove self-validation test refs from CLI/Configuration requirements; fix results file directory creation Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/0c41f310-04ab-4dc1-8f1e-c297f19312b5 Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * refactor: use TestDirectory helper in Validation_Run_WithResultsFileInNewDirectory_CreatesDirectory test Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/4b5d1e5d-5412-46c0-a53c-0dc783a2cf7f Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> Co-authored-by: Malcolm Nixon <Malcolm.nixon@gmail.com>
Add 'none' evidence source to project setup (#29) * Initial plan * Add 'none' evidence source type that returns an empty index Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/75a0f826-6b34-4a3a-b44e-45546b78f006 * Complete 'none' evidence source: tests, docs, requirements Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/a8d3447c-9cdf-471f-9314-e21a1eb99b7d * Update src/DemaConsulting.ReviewMark/ReviewMarkConfiguration.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Address review feedback: update guide.md intro and README comment for 'none' source Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/729531fc-fd47-41a4-ae10-68d50821e6f1 * Update src/DemaConsulting.ReviewMark/ReviewMarkConfiguration.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> Co-authored-by: Malcolm Nixon <Malcolm.nixon@gmail.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
feat: include review-set title in --elaborate Markdown output (#22) * Initial plan * feat: include review-set title in --elaborate Markdown output Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * fix: assert full Title table row in elaborate self-validation test Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Rename self-validation tests for clarity (#18) * Initial plan * refactor: rename self-validation tests for clarity Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Fix Sonar issues: compile-time regex (SYSLIB1045) and cognitive compl… …exity (S3776) (#17) * Initial plan * fix: resolve Sonar SYSLIB1045 and S3776 issues Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Fix Sonar code smell issues in source and test files (#14) * Initial plan * Fix Sonar code smell issues: constant extraction, collection init, and assertion improvements Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * Update test/DemaConsulting.ReviewMark.Tests/ReviewMarkConfigurationTests.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update test/DemaConsulting.ReviewMark.Tests/GlobMatcherTests.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update test/DemaConsulting.ReviewMark.Tests/GlobMatcherTests.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Replace Assert.IsTrue/IsFalse Contains wrappers with Assert.Contains/DoesNotContain in GlobMatcherTests.cs Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> Co-authored-by: Malcolm Nixon <Malcolm.nixon@gmail.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>