-
Notifications
You must be signed in to change notification settings - Fork 6
MSRV 1.85 and Rust edition 2024 #77
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…y with edition 2024 Co-authored-by: hombit <[email protected]>
Co-authored-by: hombit <[email protected]>
Co-authored-by: hombit <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR updates the minimum supported Rust version (MSRV) from 1.67 to 1.85 and upgrades the Rust edition from 2021 to 2024 across all three crates in the workspace. The MSRV bump enables the use of cargo's MSRV-aware dependency resolver and removes previous version constraints on cxx and cxx-build dependencies that were needed for MSRV 1.67/1.70 compatibility.
Key changes:
- Bumped all crate versions from 0.4.x to 0.5.0 (breaking change)
- Updated rust-version field to 1.85.0 and edition to 2024 in all Cargo.toml files
- Removed version upper bounds on
cxxandcxx-builddependencies, allowing latest 1.x versions
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| Cargo.toml | Updated main crate to version 0.5.0, edition 2024, and rust-version 1.85.0 |
| ceres-solver-sys/Cargo.toml | Updated to version 0.5.0, edition 2024, rust-version 1.85.0; removed cxx version constraints and cmake build dependency |
| ceres-solver-src/Cargo.toml | Updated to version 0.5.0, edition 2024, and rust-version 1.85.0 |
| src/solver.rs | Automatic import reordering (alphabetical) due to Rust 2024 edition formatting |
| .github/workflows/test.yml | Updated CI test matrix to use Rust 1.87 instead of 1.67 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.
Bump minimum supported Rust version to 1.85 and Rust edition to 2024, which would allow us to benefit from msrv-aware dependency resolver with cargo