Skip to content
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

Merged
merged 4 commits into from
Jun 17, 2024
Merged

cargo: update msrv to 1.75 #1084

merged 4 commits into from
Jun 17, 2024

Conversation

prestist
Copy link
Contributor

Additionally run cargo update to update all dep's.

@prestist prestist added the skip-notes This PR does not need release notes label Jun 14, 2024
@prestist
Copy link
Contributor Author

prestist commented Jun 14, 2024

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

@prestist prestist enabled auto-merge June 14, 2024 20:55
@prestist prestist force-pushed the update-MSRV branch 2 times, most recently from f988fdc to 1d5863f Compare June 14, 2024 21:28
@prestist prestist changed the title cargo:update MSRV to 1.75 cargo: update msrv to 1.75 Jun 14, 2024
Copy link

@tormath1 tormath1 left a 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)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
![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)

Copy link
Contributor Author

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.
@prestist prestist removed the skip-notes This PR does not need release notes label Jun 17, 2024
Copy link
Member

@travier travier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@prestist prestist merged commit 39c4c05 into coreos:main Jun 17, 2024
10 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.

None yet

3 participants