Skip to content

Codacy: tidy code and add coverage upload - #5

Merged
davidnmbond merged 2 commits into
mainfrom
codacy/raise-grade-to-a
Oct 5, 2026
Merged

davidnmbond merged 2 commits into
mainfrom
codacy/raise-grade-to-a

Conversation

@davidnmbond

Copy link
Copy Markdown
Contributor

What

  • Seal internal classes Application and Configuration, add XML docs to the ExecuteAsync override and GetNetworksAsync, fix catch( spacing and indentation, use a collection expression.
  • CI: add a test-with-coverage step and a Codacy coverage upload using codacy/codacy-coverage-reporter-action@v1.3.0. Both steps are skipped when no *Tests.csproj exists or when the token is absent.

Codacy numbers

Not available. The Codacy API token could not be fetched (credential broker access was denied in this session), so these fixes are based on reading the code, not on Codacy's reported issues. Before/after numbers are unknown.

Notes

  • CODACY_PROJECT_TOKEN must be set as a repository secret for coverage upload to run.
  • The repo has no tests, so coverage stays empty until a test project is added. That may be what holds the grade at B.
  • No exclusions were added.
  • Build: 0 warnings, 0 errors.

🤖 Generated with Claude Code

davidnmbond and others added 2 commits October 5, 2026 00:08
Seal internal classes, document the ExecuteAsync override and GetNetworksAsync,
fix catch spacing and indentation, use a collection expression.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

AI Reviewer: run a review on demand. To trigger the first review automatically, go to your organization or repository integration settings. AI can make mistakes. Always validate suggestions.

Run reviewer

TIP This summary will be updated as you push new changes.

@davidnmbond
davidnmbond merged commit 550ed4d into main Oct 5, 2026
5 checks passed
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.

1 participant