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

doc: Version update from 1.72.0 to 1.74.0 (released 2023-11-16) #244

Closed

Conversation

NathanielPrescott
Copy link

After reviewing the documentation I found that 1.72.0 is still listed as the MSRV, which is 3 months behind the latest version within the 6 month MSRV limit. Updating will keep the rolling MSRV statement true, add stabilized features from 1.73 + 1.74, and add internal Rust changes from 1.73.

https://doc.rust-lang.org/beta/releases.html#version-1740-2023-11-16

After updating the version I ran:
cargo clean
cargo build
cargo fix
cargo test

I found no further changes or issues that I could see needed to be made.

@drcaramelsyrup
Copy link
Contributor

Wondering if there is a particular issue that motivates this change?

The MSRV policy is simply stating that we will accept new MSRVs as long as they are at least 6 months old. We're usually happy to bump the MSRV if new features or fixes need the new Rust updates, however we try not to do so eagerly in the spirit of minimizing breaking changes.

@drcaramelsyrup drcaramelsyrup added documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file labels May 22, 2024
@NathanielPrescott
Copy link
Author

I was mostly updating this to keep the MSRV up to date with the latest acceptable Rust version. As far as I am aware the Rust language has great backwards compatibility support which should allow for easy version updates without much if any code changes. My goal was just to push the versioning forward, if anything I could pivot this PR to change the wording of the 'Rust Version' section to better state that the "MSRV should not be changed unless a version provides necessary upgrades and/or performance improvements".

After running through the benches the version update doesn't noticeably improve any performance. There was a LLVM update in 1.73.0 which may affect some minor performance (not that I see any from the benches). If anything it may help build times by a couple seconds, though this will likely be actually improved later this year with the multi-threaded frontend changes coming to the compiler.

@NathanielPrescott
Copy link
Author

After reading more about the MSRV and how it is used for libraries I understand its purposed better. That being said I believe that the version shouldn't be changed from 1.72.0 and I'll close this PR. Sorry for any wasted time on this @drcaramelsyrup

@NathanielPrescott NathanielPrescott deleted the version-1740 branch May 23, 2024 19:29
@drcaramelsyrup
Copy link
Contributor

Not a problem, we appreciate the enthusiasm!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants