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: upgrade rust-version to 1.65.0 #7993
Conversation
- GATs are finally here! I'd be surprised if we cannot make use of it. - stabilized `std::backtrace::Backtrace`, we should see if we can remove our `backtrace` dependency now - let else statements are now a thing
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.
Yes, lets remove bt, but lets' do it in a serparate PR (better to split "do upgrade" and "take advantage of an upgrade" in two PRs).
I bet we could make use of let else
in a bunch of places as well.
@@ -4,7 +4,7 @@ version = "0.0.0" | |||
publish = false | |||
authors = ["Near Inc <hello@nearprotocol.com>"] | |||
# Please update rust-toolchain.toml as well when changing version here: | |||
rust-version = "1.64.0" | |||
rust-version = "1.65.0" |
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.
drive by, but specifying rust-version for test contracts doesnt' really makes sense, these crates shond basically never be published to crates.io
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.
would you suggest to just leave it unspecified? i.e. just delete this line?
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.
Yeah, that'd be better, but we also need to make sure that themis
doesn't complain.
- GATs are finally here! I'd be surprised if we cannot make use of it. - stabilized `std::backtrace::Backtrace`, we should see if we can remove the `backtrace` dependency - let else statements are now a thing
- GATs are finally here! I'd be surprised if we cannot make use of it. - stabilized `std::backtrace::Backtrace`, we should see if we can remove the `backtrace` dependency - let else statements are now a thing
https://blog.rust-lang.org/2022/12/15/Rust-1.66.0.html Updated the same files as in the previous version bump: #7993 This enables #7650 (see [this comment](#7650 (comment))). Also includes fixes for the following warning: ``` warning: for loop over an `Option`. This is more readably written as an `if let` statement ... = note: `#[warn(for_loops_over_fallibles)]` on by default ```
std::backtrace::Backtrace
, we should see if we can removethe
backtrace
dependency