Merged
Conversation
This PR addresses issue #48 by re-enabling code coverage reporting using GitHub Actions and Go's native coverage tools, without requiring external SaaS services like Codecov or Coveralls. The blocking Go issue (golang/go#35781) causing "inconsistent NumStmt" errors has been worked around in commit 7ad8c5d by removing the -race flag from coverage runs. Changes: - Replace Codecov integration with GitHub Actions native coverage - Extract and display coverage percentage in workflow summary - Add optional badge generation using GitHub gist (no external services) - Update documentation to reflect new coverage approach - Coverage generation works successfully (83.8% overall coverage) Benefits of this approach: - No external SaaS dependencies or API tokens required - Coverage data visible in GitHub Actions workflow summary - Uses Go's built-in coverage tools - Badge generation is optional (requires GIST_SECRET if desired) - Follows modern best practices for Go projects in 2024-2025 Fixes #48
osteele
deleted the
claude/code-coverage-review-011CUv7fxXMxPcMpyhW7CYuY
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR addresses issue #48 by re-enabling code coverage reporting using GitHub Actions and Go's native coverage tools, without requiring external SaaS services like Codecov or Coveralls.
The blocking Go issue (golang/go#35781) causing "inconsistent NumStmt" errors has been worked around in commit 7ad8c5d by removing the -race flag from coverage runs.
Changes:
Benefits of this approach:
Fixes #48
Checklist
make testpasses.make lintpasses.