Skip to content

2024 edition #355

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

2024 edition #355

wants to merge 10 commits into from

Conversation

ludfjig
Copy link
Contributor

@ludfjig ludfjig commented Mar 13, 2025

Updates rust edition to 2024. A downside is that this raises minimum version to 1.85 for our next hyperlight release (0.3.0).

TODO: For some reason, our signal handler stops working, with the following rust error message
fatal runtime error: failed to initiate panic, error 3. This is unrelated to edition change, and is introduced going from rust 1.83->1.84. The error only occurs in release mode.

@ludfjig ludfjig added the area/infrastructure Concerns infrastructure rather than core functionality label Mar 13, 2025
@ludfjig ludfjig force-pushed the 2024_edition branch 2 times, most recently from 763ec2d to 4ae3754 Compare March 13, 2025 22:50
ludfjig added 10 commits March 13, 2025 16:01
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
area/infrastructure Concerns infrastructure rather than core functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant