Lint items after statements in local macro expansions #6176
Conversation
The items_after_statements lint was skipping all expansions. Instead we should still lint local macros. Fixes rust-lang#578
|
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @phansch (or someone else) soon. If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes. Please see the contribution instructions for more information. |
rust-highfive
added
the
S-waiting-on-review
achris
changed the title
|
@bors r+ thanks! |
|
📌 Commit 32e2021 has been approved by |
|
☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test |
The items_after_statements lint was skipping all expansions. Instead
we should still lint local macros.
Fixes #578
Please keep the line below
changelog: The items_after_statements now applies to local macro expansions