Skip to content

Conversation

@Niki4tap
Copy link
Contributor

@Niki4tap Niki4tap commented Jan 30, 2023

Fixes #10259

changelog: FP: [multiple_unsafe_ops_per_block]: No longer lints in external macros
#10260

@rustbot
Copy link
Collaborator

rustbot commented Jan 30, 2023

r? @xFrednet

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jan 30, 2023
@xFrednet
Copy link
Contributor

How the turns have tabled, I just addressed your review in marker ^^. Thank you for the PR, the changes look good to me :)

@bors r+

@bors
Copy link
Contributor

bors commented Jan 30, 2023

📌 Commit 926c5e4 has been approved by xFrednet

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Jan 30, 2023

⌛ Testing commit 926c5e4 with merge d020fd7...

@bors
Copy link
Contributor

bors commented Jan 30, 2023

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: xFrednet
Pushing d020fd7 to master...

@bors bors merged commit d020fd7 into rust-lang:master Jan 30, 2023
@Niki4tap Niki4tap deleted the external_macro_fp branch January 30, 2023 20:28
@Niki4tap
Copy link
Contributor Author

Thanks for the quick review @xFrednet!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties

Projects

None yet

Development

Successfully merging this pull request may close these issues.

multiple_unsafe_ops_per_block should not warn external macros

4 participants