Skip to content

ci: report Rust coverage to Codecov - #204

Merged
lan17 merged 1 commit into
mainfrom
codex/rust-codecov
Sep 22, 2026
Merged

lan17 merged 1 commit into
mainfrom
codex/rust-codecov

Conversation

@lan17

@lan17 lan17 commented Sep 22, 2026

Copy link
Copy Markdown
Owner

Rust CI runs the native, conformance, and real-server tests but does not publish a coverage report. This instruments those existing tests with pinned cargo-llvm-cov tooling and uploads their combined LCOV report to Codecov under the rust flag on pull requests and main.

TypeScript and Go now upload on pull requests as well as main, labeled typescript and go, so Codecov receives all three language reports for the same commit. This preserves the Go coverage collection added on main. All uploads use the existing pinned Codecov action and GitHub OIDC. The Rust LCOV file is also retained as a GitHub Actions artifact for 14 days.

The Rust job retains formatting, clippy, package verification, native tests, committed smoke conformance, the Redis documentation scenario, and Redis/Valkey/Cluster integrations. Package verification runs before instrumentation. Standalone tests/examples, toolchain sources, and the optional test utility runtime are excluded from the report; inline unit-test modules follow LLVM's normal coverage semantics. Doctests still execute, but their coverage is not collected on stable Rust. Full generated conformance remains its separate validation lane.

Validation:

  • Ran the instrumented Make/Cargo test sequence locally: formatting, clippy, 155 native tests, 1,738 conformance cases, all five Docker integration tests, and the Redis documentation test passed.
  • Exported and inspected LCOV, confirming it contains only the 27 intended Rust source files. The local report records 5,950 of 6,310 executable source lines hit (94.29%); hosted Codecov will establish the CI baseline.
  • Verified Rust baseline at 3415e84: 94.29% (5,950/6,310 lines across 27 files). The Rust source and instrumentation are unchanged by the rebase onto Go coverage support.
  • Actionlint and git diff --check passed.
  • Independent workflow review found no actionable issues.

No Rust implementation or formal specification changes.

— Levicus 🤖

@codecov-commenter

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@lan17
lan17 force-pushed the codex/rust-codecov branch from 3415e84 to 05f92d1 Compare September 22, 2026 17:59
@lan17
lan17 merged commit a966127 into main Sep 22, 2026
19 checks passed
@lan17
lan17 deleted the codex/rust-codecov branch September 22, 2026 18:03
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.

2 participants