Skip to content

fix ci checks#5

Merged
jianjian01 merged 5 commits into
cq-lib:developfrom
jianjian01:chore/fix-ci-checks
Jul 1, 2026
Merged

fix ci checks#5
jianjian01 merged 5 commits into
cq-lib:developfrom
jianjian01:chore/fix-ci-checks

Conversation

@jianjian01

Copy link
Copy Markdown
Collaborator

No description provided.

Fix Clippy failures across core tests and Python bindings so the workspace passes with `-D warnings`.

- Replace mechanical lint patterns in tests such as bool comparisons, needless range loops, redundant closures, unnecessary clones/casts, and  assign-op patterns
- Avoid approximate PI constants in tests where arbitrary numeric literals are  sufficient
- Move Python standard gate tests after all items to satisfy item ordering lint
- Allow high-arity Python visualization wrappers locally to preserve the public Python API
- Format affected visualization layout code
Upgrade pyo3 and numpy to 0.29 to resolve RustSec advisories for pyo3 0.27. Update Cargo.lock with compatible dependency versions, including memmap2 0.9.11.
Explicitly link the native libraries required by the Rust staticlib when building the C example through CMake.

- Use the C++ linker and link libc++ on macOS for Rust/libc++ symbols
- Link ntdll and Rust std related system libraries on Windows MSVC
- Keep Linux native library linkage scoped to non-Apple Unix platforms
Add explicit `from_py_object` or `skip_from_py_object` options to Clone-backed Python binding classes for PyO3 0.29.

Types that are accepted as owned Python API arguments or extracted via `FromPyObject` keep the previous behavior with `from_py_object`. Types that are only returned, borrowed, or used as Python-side handles now use `skip_from_py_object` to avoid unnecessary implicit extraction and clone  semantics.

This resolves the PyO3 deprecation warnings promoted by `-D warnings` under clippy.
@jianjian01 jianjian01 merged commit 7e2af02 into cq-lib:develop Jul 1, 2026
8 of 15 checks passed
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