atharvalade opened a new issue, #2866: URL: https://github.com/apache/iggy/issues/2866
The Python SDK coverage reports 0% in Codecov because the current `cargo-llvm-cov` integration has three flaws: manual instrumentation instead of `cargo llvm-cov show-env`, missing `CARGO_TARGET_DIR` centralization for the monorepo path dependency on `core/sdk`, and LCOV source paths relative to the crate root (`src/client.rs`) instead of the repo root (`foreign/python/src/client.rs`) which Codecov's `python` flag expects. This PR rewrites both the post-merge and pre-merge coverage pipelines following the proven approach from [cjermain/rust-python-coverage](https://github.com/cjermain/rust-python-coverage) and the existing Rust coverage job pattern already in `post-merge.yml`. Key changes: use `source <(cargo llvm-cov show-env --export-prefix)` with `CARGO_TARGET_DIR=$CARGO_LLVM_COV_TARGET_DIR`, replace raw `llvm-profdata`/`llvm-cov` with `cargo llvm-cov report --lcov`, fix paths with `sed`, filter dependency code via `--ignore-filename-regex`, drop `--release` from `maturin develop`, and add `LCOV_EXCL_START`/`LCOV_EXCL_STOP` markers around the `#[pymodule]` init function in `lib.rs`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
