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

Rename PanicInfo to PanicHookInfo #6865

Merged
merged 2 commits into from
Dec 13, 2024
Merged

Rename PanicInfo to PanicHookInfo #6865

merged 2 commits into from
Dec 13, 2024

Conversation

athei
Copy link
Member

@athei athei commented Dec 12, 2024

Starting with Rust 1.82 PanicInfo is deprecated and will throw warnings when used. The new type is available since Rust 1.81 and should be available on our CI.

@athei athei added the R0-silent Changes should not be mentioned in any release notes label Dec 12, 2024
@athei athei requested a review from koute as a code owner December 12, 2024 21:25
@athei
Copy link
Member Author

athei commented Dec 12, 2024

/cmd prdoc --audience runtime_dev --bump=fix

Copy link
Contributor

Command "prdoc --audience runtime_dev --bump=fix" has failed ❌! See logs here

@athei
Copy link
Member Author

athei commented Dec 12, 2024

/cmd prdoc --audience node_dev --bump patch

@bkchr bkchr requested a review from a team December 13, 2024 07:24
@bkchr bkchr enabled auto-merge December 13, 2024 07:24
@bkchr bkchr added this pull request to the merge queue Dec 13, 2024
Merged via the queue into master with commit 9ce80f6 Dec 13, 2024
200 of 201 checks passed
@bkchr bkchr deleted the at/fix-warning branch December 13, 2024 09:40
dudo50 pushed a commit to paraspell-research/polkadot-sdk that referenced this pull request Jan 4, 2025
Starting with Rust 1.82 `PanicInfo` is deprecated and will throw
warnings when used. The new type is available since Rust 1.81 and should
be available on our CI.

---------

Co-authored-by: command-bot <>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants