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 rustc v1.75 #429

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

tnull
Copy link
Collaborator

@tnull tnull commented Jan 13, 2025

Closes #427.

Recently, rustls bumped their MSRV to 1.71. As we depend on them and don't want to continuously pin this security-critical dependency back, we're looking to bump our MSRV to a version >= 1.71, too.

Here, we propose to bump the MSRV to rustc version 1.75, which was chosen because:

a) it's > 1 year old
b) it provides a buffer to 1.71, i.e., if some crate bumped to a version > 1.71, there is a chance we don't immediately have to react again
c) it stabilized async fns in traits (see
https://blog.rust-lang.org/2023/12/21/async-fn-rpit-in-traits.html), which might become handy for related crates.

Recently, `rustls` bumped their MSRV to 1.71. As we depend on them and
don't want to continuously pin this security-critical dependency back,
we're looking to bump our MSRV to a version >= 1.71, too.

Here, we propose to bump the MSRV to rustc version 1.75, which was
chosen because:

a) it's > 1 year old
b) it provides a buffer to 1.71, i.e., if some crate bumped to a version
> 1.71, there is a chance we don't immediately have to react again
c) it stabilized `async fn`s in traits (see
https://blog.rust-lang.org/2023/12/21/async-fn-rpit-in-traits.html),
which might become handy for related crates.
@tnull tnull force-pushed the 2025-01-bump-msrv-to-1.75 branch from 4a574d0 to a98d756 Compare January 13, 2025 10:07
@tnull tnull requested review from jkczyz and arik-so and removed request for jkczyz January 15, 2025 08:42
@tnull
Copy link
Collaborator Author

tnull commented Jan 15, 2025

CI failures are unrelated and fixed as part of #426.

@arik-so
Copy link

arik-so commented Jan 15, 2025

ACKing due to CI failures being unrelated.

@tnull tnull merged commit 17f28f7 into lightningdevkit:main Jan 15, 2025
0 of 26 checks passed
@tnull tnull mentioned this pull request Jan 15, 2025
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.

Figure out new MSRV target >= 1.71.0
2 participants