Skip to content

ci: publish repository coverage via codecov#19

Draft
Ramlaoui wants to merge 4 commits intomainfrom
codex/docs-readme-coverage
Draft

ci: publish repository coverage via codecov#19
Ramlaoui wants to merge 4 commits intomainfrom
codex/docs-readme-coverage

Conversation

@Ramlaoui
Copy link
Copy Markdown
Collaborator

@Ramlaoui Ramlaoui commented May 9, 2026

Summary

  • upload Python and Rust coverage reports to Codecov from CI
  • show the repository coverage badge at the top of README.md
  • remove the custom PR coverage summary script and README coverage section

Testing

  • validated .github/workflows/ci.yml parses
  • not run full project test suites locally

Notes

  • the README badge tracks the main branch coverage in Codecov
  • PR coverage updates come from Codecov uploads in the CI workflow

@Ramlaoui Ramlaoui changed the title docs: add README coverage guidance ci: automate PR coverage reporting May 9, 2026
@Ramlaoui Ramlaoui force-pushed the codex/docs-readme-coverage branch from 712bda5 to 773ab1d Compare May 9, 2026 13:08
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 9, 2026

Coverage Report

Scope Covered Lines Total Lines Line Coverage
Python 221 623 35.47%
Rust 1286 5200 24.73%

Artifacts:

  • Python XML/HTML coverage: atompack-py/coverage/
  • Rust LCOV coverage: coverage/rust.lcov

@Ramlaoui Ramlaoui changed the title ci: automate PR coverage reporting ci: publish repository coverage via codecov May 9, 2026
@Ramlaoui
Copy link
Copy Markdown
Collaborator Author

Ramlaoui commented May 9, 2026

This is blocked on Codecov onboarding, not on the coverage generation code itself.

Current status:

  • the workflow can generate both Python and Rust coverage reports
  • the repository coverage badge / PR reporting cannot be completed until LeMaterial/atompack is enabled in Codecov
  • that requires org-level GitHub App access or someone who already has Codecov org access to enable the repo and provide the upload token

Concretely, we cannot finish the Codecov path unless one of these happens:

  1. a LeMaterial org admin grants the Codecov GitHub App access to atompack
  2. the repo is already enabled in Codecov and someone provides the CODECOV_TOKEN
  3. we abandon Codecov and switch to a GitHub-only badge flow instead

Marking this PR as draft until that dependency is resolved.

@Ramlaoui Ramlaoui marked this pull request as draft May 9, 2026 18:08
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