-
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
Tweak may_dangle syntax for MCP 563 #107229
Conversation
(rustbot has picked a reviewer for you, use r? to override) |
This comment has been minimized.
This comment has been minimized.
future note: It's good practice to mark as a draft and |
7639858
to
80d730b
Compare
oh. does CI run on drafts...? we thought it didn't? but yes, noted about the |
The job Click to see the possible cause of the failure (guessed by this bot)
|
hmm, actually unsure how to proceed. gonna wait for a fix for #107231 and see what we can do from there. |
This could be fixed without changing attribute validation globally, the rust/compiler/rustc_codegen_ssa/src/codegen_attrs.rs Lines 153 to 182 in c8e6a9e
|
@SoniEx2 FYI: when a PR is ready for review, please send a message containing |
we aren't entirely sure how to do this. also waiting for feedback on rust-lang/rfcs#3390 because that happened. |
(don't mind us just using CI for development)