-
Notifications
You must be signed in to change notification settings - Fork 13k
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
Enable -W semicolon_in_expressions_from_macros
in bootstrap
#83089
Conversation
(rust-highfive has picked a reviewer for you, use r? to override) |
This comment has been minimized.
This comment has been minimized.
r=me in theory but looks like there's at least one use left |
This is now blocked on bumping the |
@Aaron1011 this should be possible since #83776 |
This comment has been minimized.
This comment has been minimized.
It looks like rust-lang/stdarch#1080 wasn't included in the submodule bump. |
This comment has been minimized.
This comment has been minimized.
Now that stdarch has been updated, we can do this without breaking the build.
@Mark-Simulacrum This is now ready to merge |
@bors r+ |
📌 Commit 020d83d has been approved by |
☀️ Test successful - checks-actions |
Now that stdarch has been updated, we can do this without breaking the
build.