-
Notifications
You must be signed in to change notification settings - Fork 11
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
Chore: Update Deps and increase MSRV to 1.70 #342
Conversation
1fe06fa
to
5642cae
Compare
I believe the MSRV increase from 1.66 to 1.70 should be safe, since we distribute precompiled binaries now. We have several dependencies, that have 1.70 (release 2023-06-01) as their MSRV. I still listed this change, in the changelog under the breaking changes section. |
Cargo.toml
Outdated
ui_test = "0.21.2" | ||
visibility = "0.1.0" | ||
yansi = "1.0.0-rc.1" | ||
yansi = "1.0.0-gamma" |
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.
1.0.0-rc.1
is a more recent one (versions)
@@ -44,9 +44,9 @@ marker_utils = { path = "./marker_utils", version = "0.5.0-dev" } | |||
bumpalo = "3.14" | |||
camino = { version = "1.1", features = ["serde1"] } | |||
cargo_metadata = "0.18" | |||
clap = { version = "=4.3.24", features = ["string", "derive"] } |
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.
I wonder why it was pinned
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.
Clap increased the minimal rust version in newer releases. The pin was me trying to keep rust 1.66 as the minimal version. Now we can just use the newest version again.
e5be5be
to
c3b900e
Compare
c3b900e
to
a745a05
Compare
Not much to say, just some prep for the next release.