Merge pull request #6 from cjermain/claude/session-011CUYo18ZqbutpJ67DM6jab

Update dependencies and achieve 100% coverage

Author: cjermain

.codecov.yml

@@ -0,0 +1,23 @@
+codecov:
+  require_ci_to_pass: yes
+
+coverage:
+  status:
+    project:
+      default:
+        target: 100%
+        threshold: 0%
+    patch:
+      default:
+        target: 100%
+
+ignore:
+  - "tests/"
+  - "**/*test*.rs"
+
+comment:
+  layout: "reach,diff,flags,tree"
+  behavior: default
+
+# Exclude specific patterns from coverage
+# coverage:off and coverage:on comments are recognized by CodeCov

.github/workflows/CI.yml

@@ -12,19 +12,16 @@ jobs:
         os: ["ubuntu"]
     runs-on: ${{ matrix.os }}-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v5
       - uses: actions/setup-python@v5
         with:
           python-version: "3.12"
       - name: Install uv
         uses: astral-sh/setup-uv@v5
         with:
           enable-cache: true
-      - uses: actions-rs/toolchain@v1
+      - uses: dtolnay/rust-toolchain@stable
         with:
-          toolchain: stable
-          override: true
-          profile: minimal
           components: llvm-tools-preview
       - name: Install cargo-llvm-cov
         uses: taiki-e/install-action@cargo-llvm-cov

Cargo.toml

@@ -9,7 +9,7 @@ name = "foobar"
 crate-type = ["cdylib"]
 
 [dependencies]
-pyo3 = { version = "0.23.3", features = ["extension-module"] }
+pyo3 = { version = "0.23.5", features = ["extension-module"] }
 
 [package.metadata.maturin]
 python-source = "python"

README.md

@@ -99,4 +99,15 @@ The last step of this process happens in the CI, where we upload both coverage
 files to CodeCov. [Merging reports](https://docs.codecov.com/docs/merging-reports)
 is an automatic feature of CodeCov, so the final view shows the combined view.
 
+# Excluding code from coverage
+
+The `#[pymodule]` initialization function in `src/lib.rs` is surrounded by
+`LCOV_EXCL_START` and `LCOV_EXCL_STOP` [exclusion markers](https://manpages.ubuntu.com/manpages/noble/man5/lcovrc.5.html).
+These markers instruct LCOV-compatible coverage tools to exclude the enclosed
+region from coverage analysis. This is necessary because PyO3's module
+initialization code is generated by a procedural macro and cannot be directly
+tested in a way that produces coverage data (see [rust-lang/rust#84605](https://github.com/rust-lang/rust/issues/84605)).
+The markers appear as comments in the source code and are recognized by CodeCov
+when processing the LCOV report generated by `cargo-llvm-cov`.
+
 https://codecov.io/gh/cjermain/rust-python-coverage

pyproject.toml

@@ -8,13 +8,13 @@ classifiers = [
     "Programming Language :: Python :: Implementation :: PyPy",
 ]
 dependencies = [
-    "pytest>=8.3.4",
+    "pytest>=8.4.2",
     "pytest-cov>=6.0.0",
-    "maturin>=1.8.1",
+    "maturin>=1.9.6",
 ]
 
 [build-system]
-requires = ["maturin>=1.0,<2.0"]
+requires = ["maturin>=1.9,<2.0"]
 build-backend = "maturin"
 
 [tool.maturin]

src/lib.rs

@@ -13,11 +13,13 @@ fn pyo3_sum(a: usize, b: usize) -> PyResult<usize> {
 }
 
 /// A Python module implemented in Rust.
+// LCOV_EXCL_START - PyO3 module initialization cannot be directly tested (see rust-lang/rust#84605)
 #[pymodule]
 fn _foobar(_py: Python, m: &Bound<'_, PyModule>) -> PyResult<()> {
     m.add_function(wrap_pyfunction!(pyo3_sum, m)?)?;
     Ok(())
 }
+// LCOV_EXCL_STOP
 
 #[cfg(test)]
 mod tests {