-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Expand box_vec lint to box_collection #7693
Conversation
r? @Manishearth (rust-highfive has picked a reviewer for you, use r? to override) |
@flip1995 it seems that |
@F3real you don't have to remove mentions, you just need to make sure the |
@bors r+ |
📌 Commit bb971e0 has been approved by |
Expand box_vec lint to box_collection fixed #7451 changelog: Report unnecessary boxing for collections
💔 Test failed - checks-action_test |
@bors retry |
☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test |
fixed #7451
changelog: Expand
box_vec
into [box_collection
], and have it error on all sorts of boxed collections