Skip to content

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 arik-so and jkczyz 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