Elioooon commented on PR #4017: URL: https://github.com/apache/iggy/pull/4017#issuecomment-5583532707
The newly approved CI run caught a formatting regression in my branch after the upstream merge: replacing error closures with `to_runtime_error` left 11 expressions in `foreign/python/src/client.rs` that Rust 1.98 rustfmt reformats. The Python lint job failed at `cargo fmt`; Ruff passed, and Clippy/pyrefly were not reached. The Python test job was cancelled, so this run does not establish integration-test success. Fixed in `2969bc30c` with rustfmt only. Local verification: ```text cargo fmt --manifest-path foreign/python/Cargo.toml -- --check cargo clippy --manifest-path foreign/python/Cargo.toml --all-targets --all-features --offline -- -D warnings cargo test --manifest-path foreign/python/Cargo.toml --lib --offline # 3 passed ``` Formatting and Clippy pass with the pinned Rust 1.98.0 toolchain. No API or behavior changes; I did not rerun the server-backed Python integration suite for this formatting-only follow-up. The new head still needs CI verification. -- 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]
