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

Bump MSRV to 1.66.0 #450

Merged
merged 4 commits into from
May 5, 2023
Merged

Bump MSRV to 1.66.0 #450

merged 4 commits into from
May 5, 2023

Conversation

cgwalters
Copy link
Member

@cgwalters cgwalters commented Mar 21, 2023

Bump MSRV to 1.66.0

This is what we have elsewhere in coreos/ and I think will
be prep for further PRs.


Allow clippy::box_default

Because this code is right, and what cargo clippy --fix wants
to do actually breaks compilation.


tree-wide: Fix various minor clippy lints

The unnecessary borrows one always feels like the most churn
for the least value, but let's do it.


filetree: Use .write_all() in tests

This clippy lint catches a real bug.


This is what we have elsewhere in coreos/ and I think will
be prep for further PRs.

Signed-off-by: Colin Walters <walters@verbum.org>
Because this code is right, and what `cargo clippy --fix` wants
to do actually breaks compilation.

Signed-off-by: Colin Walters <walters@verbum.org>
The unnecessary borrows one always feels like the most churn
for the least value, but let's do it.

Signed-off-by: Colin Walters <walters@verbum.org>
This clippy lint catches a real bug.

Signed-off-by: Colin Walters <walters@verbum.org>
@jmarrero jmarrero merged commit 25bd46c into coreos:main May 5, 2023
6 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

2 participants