-
Notifications
You must be signed in to change notification settings - Fork 105
Bump rust edition to 2024, rust version to 1.85 #515
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
Conversation
|
Arrow-rs is bumped to edition 2024, and 1.85 is now used by 90% of downloads based on stats from https://lib.rs/stats. I believe it's fair to bump to 2024 now. Another advantage of edition 2024 is that raising the MSRV is no longer a breaking change, making this a one shot solution. |
|
good points, I'll have that later today/tomorrow |
|
Ok pushed it, bumped msrv to 1.85 and the rust edition to 2024. |
|
most of the change is just the new rutfmt ordering, other than that its just a few clippy warnings |
alamb
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this increases the MSRV woudl that make it a breaking change (aka we can only release this as part of a major release)?
|
IIRC MSRV updates are considered to be "non-breaking", though I can't recall a canonical source for that. |
Here is one source: Though it has a lot of discussion which I didn't read |
|
Yes, it's considered a non-breaking change. But only after edition 2024 it becomes a really non-harmful thing thanks to https://doc.rust-lang.org/edition-guide/rust-2024/cargo-resolver.html So my suggestion is that we still treat this change as a breaking one and after this PR, we can bump MSRV freely. |
|
So shall we merge this one in and plan to cut a new major release of object_store? |
|
Should I also remove the |
|
It would be great if we could remove that section |
843dd72 to
1495da2
Compare
|
Sounds great -- I will plan on making a new release in Nov
|
Which issue does this PR close?
Unlocks #514.
Rationale for this change
90% of downloads use 1.85, and 2024 unlocks future msrv bumps.
What changes are included in this PR?
Bumping rust edition and msrv.
Are there any user-facing changes?
Increasing the msrv might have an impact on some dependencies and users, but I assume the vast majority of them will be fine with upgrading (For example - DataFusion is at 1.86 and arrow-rs is at 1.85).