Skip to content

Conversation

@mkeeter
Copy link
Contributor

@mkeeter mkeeter commented Oct 23, 2025

I got tired of my editor's LSP complaining about gen, and decided to update everything to Rust 2024.

This was mostly mechanical renaming of gengeneration, Clippy fixes, and rustfmt.

VolumeConstructionRequest has to have a #[serde(rename = "gen")] annotation, because it's been stored on disk.

@mkeeter mkeeter requested review from jmpesp and leftwo October 23, 2025 20:38
Copy link
Contributor

@leftwo leftwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Once 17 lands, we can send this in.

I can't say I read every single line, but I did scroll slowly through all the changes.

@mkeeter mkeeter merged commit 2481b77 into main Nov 5, 2025
17 checks passed
@mkeeter mkeeter deleted the mkeeter/rust-2024 branch November 5, 2025 19:02
mkeeter added a commit that referenced this pull request Nov 6, 2025
#1799 renamed `gen` → `generation`, because the former is a reserved
word in Rust 2024. That PR special-cased the `generation` field in the
volume construction request, because it's serialized to disk; however,
we should also special-case CLI arguments to avoid breaking existing
scripts.
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.

3 participants