Skip to content

ci: add Codecov test analytics#51

Merged
JohnRDOrazio merged 2 commits intodevfrom
chore/codecov-test-analytics
Apr 9, 2026
Merged

ci: add Codecov test analytics#51
JohnRDOrazio merged 2 commits intodevfrom
chore/codecov-test-analytics

Conversation

@JohnRDOrazio
Copy link
Copy Markdown
Member

@JohnRDOrazio JohnRDOrazio commented Apr 9, 2026

Summary

  • Adds --junitxml=junit.xml to pytest to generate JUnit XML test results
  • Uploads test results to Codecov via codecov/test-results-action@v1
  • Uses if: ${{ !cancelled() }} so results are uploaded even on test failures

Test plan

  • CI passes and test results appear on Codecov dashboard

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Added Codecov coverage badge to the README.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 9, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

These changes integrate Codecov test results reporting into the release workflow by adding JUnit XML output to pytest and a new Codecov test results action, while adding a Codecov coverage badge to the README.

Changes

Cohort / File(s) Summary
CI/Workflow Configuration
.github/workflows/release.yml
Added --junitxml=junit.xml flag to pytest invocation and introduced codecov/test-results-action@v1 step with conditional execution to report test results to Codecov.
Documentation
README.md
Added Codecov coverage badge linking to the repository's Codecov metrics page.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 A badge now shines, so bright and true,
Codecov's eye sees all we do,
Test results flow with JUnit's might,
Our coverage measured, crystal bright! 🌟

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the primary change: adding Codecov test analytics integration to the CI workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/codecov-test-analytics

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@JohnRDOrazio JohnRDOrazio merged commit 29ab902 into dev Apr 9, 2026
10 of 11 checks passed
@JohnRDOrazio JohnRDOrazio deleted the chore/codecov-test-analytics branch April 9, 2026 02:27
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