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

Set Rust edition to 2021 #101

Merged
merged 2 commits into from
Jun 3, 2024
Merged

Set Rust edition to 2021 #101

merged 2 commits into from
Jun 3, 2024

Conversation

faern
Copy link
Member

@faern faern commented May 29, 2024

No immediate need for the new edition. But no reason to leave it at 2018. Allows more modern Rust code and can be helpful in certain cases. Uses newer dependency resolver, brings more stuff into the prelude and more.


This change is Reviewable

@faern faern requested a review from Serock3 May 29, 2024 20:49
@faern faern force-pushed the upgrade-to-rust-2021 branch from 8b5eec2 to 36aaea5 Compare May 29, 2024 21:45
Copy link

@Serock3 Serock3 left a comment

Choose a reason for hiding this comment

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

Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

faern added 2 commits June 4, 2024 00:28
No immediate need for the new edition. But no reason to leave it at
2018. Allows more modern Rust code and can be helpful in certain cases
@faern faern force-pushed the upgrade-to-rust-2021 branch from 36aaea5 to fddc4ca Compare June 3, 2024 22:28
@faern faern merged commit 56b4835 into main Jun 3, 2024
7 of 8 checks passed
@faern faern deleted the upgrade-to-rust-2021 branch June 3, 2024 22:32
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.

2 participants