-
Notifications
You must be signed in to change notification settings - Fork 106
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
cargo: update msrv to 1.75 #1084
Conversation
Due to nightly not compiling it looks like we have a chicken/egg issue with the lint-ing. I am going to update this pr to include the change from this, and deal with the fallout from the merge of this change later. coreos/repo-templates#230 |
f988fdc
to
1d5863f
Compare
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.
LGTM.
README.md
Outdated
@@ -1,7 +1,7 @@ | |||
# Afterburn | |||
|
|||
[![crates.io](https://img.shields.io/crates/v/afterburn.svg)](https://crates.io/crates/afterburn) | |||
![minimum rust 1.71](https://img.shields.io/badge/rust-1.71%2B-orange.svg) | |||
![minimum rust 1.75](https://img.shields.io/badge/rust-1.71%2B-orange.svg) |
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.
![minimum rust 1.75](https://img.shields.io/badge/rust-1.71%2B-orange.svg) | |
![minimum rust 1.75](https://img.shields.io/badge/rust-1.75%2B-orange.svg) |
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.
Ah good catch! Thank you
Additionally run `cargo update` to update all dep's.
I have made a MR into repo-templates to update this file. seen here: coreos/repo-templates#230 Once it lands the downstream MR for updating the toolchain will not be able to merge due to failing CI we currently have, and we cannot merge the fix for the failing CI without it. So add the lint CI update here to allow CI fixes to be mergable.
First: update default_features to default-features in cargo.toml Second: replace get(0) with first()
Apprently we have had deadcode which is now being picked up by latest MSRV lint. Created issue to evaluate it coreos#1085.
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.
LGTM
Additionally run
cargo update
to update all dep's.