Skip to content

Update to 1.85 and Rust Edition 2024 #534

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

Merged
merged 12 commits into from
May 29, 2025

Conversation

simongdavies
Copy link
Contributor

See individual commit messages for details.

I retained rustfmt version as 2021, we should update this in a distinct PR as it causes updates to over 80 files

@simongdavies simongdavies added the kind/enhancement For PRs adding features, improving functionality, docs, tests, etc. label May 28, 2025
@simongdavies simongdavies added the area/infrastructure Concerns infrastructure rather than core functionality label May 28, 2025
@simongdavies simongdavies mentioned this pull request May 28, 2025
Copy link
Contributor

@jprendes jprendes left a comment

Choose a reason for hiding this comment

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

LGTM so far.
Left a few comments, and there are still some clippy issues in CI.

@marosset
Copy link
Contributor

I'll update the dev images to install 1.85.0

Copy link
Contributor

@ludfjig ludfjig left a comment

Choose a reason for hiding this comment

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

LGTM, but agree that most expr_2021 can be changed back to expr.

NOTE: I attempted to run flatc to re-generate the flatbuffer generated files but it did not do the updates for 2024 edition.

Updated manually for review feedback
Signed-off-by: Simon Davies <[email protected]>
Signed-off-by: Simon Davies <[email protected]>
The is mainly the result of running `cargo fix --edition` along with a couple of manula changes to fix clippy "shared reference to mutable static" warnings

Signed-off-by: Simon Davies <[email protected]>
running rustfmt with edition 2024 creates a lot of noise

Signed-off-by: Simon Davies <[email protected]>
Signed-off-by: Simon Davies <[email protected]>
Signed-off-by: Simon Davies <[email protected]>
Copy link
Contributor

@jprendes jprendes left a comment

Choose a reason for hiding this comment

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

LGTM, just some nits, feel free to ignore them

@simongdavies
Copy link
Contributor Author

@jprendes I am going to merge this and follow up with the nits later

@simongdavies simongdavies merged commit 3f6e9b9 into hyperlight-dev:main May 29, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/infrastructure Concerns infrastructure rather than core functionality kind/enhancement For PRs adding features, improving functionality, docs, tests, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants