Skip to content
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

Merged
merged 3 commits into from
Sep 22, 2021
Merged

Expand box_vec lint to box_collection #7693

merged 3 commits into from
Sep 22, 2021

Conversation

F3real
Copy link
Contributor

@F3real F3real commented Sep 20, 2021

fixed #7451

changelog: Expand box_vec into [box_collection], and have it error on all sorts of boxed collections

@rust-highfive
Copy link

r? @Manishearth

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Sep 20, 2021
@F3real
Copy link
Contributor Author

F3real commented Sep 20, 2021

@flip1995 it seems that remark failure is related to mention of box_vec in CHANGELOG.md. Should we also remove mentions from changelog? It feels a bit weird.

@Manishearth
Copy link
Member

@F3real you don't have to remove mentions, you just need to make sure the box_vec link reference exists and links to box_collections

@Manishearth
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Sep 22, 2021

📌 Commit bb971e0 has been approved by Manishearth

@bors
Copy link
Contributor

bors commented Sep 22, 2021

⌛ Testing commit bb971e0 with merge 5ed0155...

bors added a commit that referenced this pull request Sep 22, 2021
Expand box_vec lint to box_collection

fixed #7451

changelog:
Report unnecessary boxing for collections
@bors
Copy link
Contributor

bors commented Sep 22, 2021

💔 Test failed - checks-action_test

@Manishearth
Copy link
Member

@bors retry

@bors
Copy link
Contributor

bors commented Sep 22, 2021

⌛ Testing commit bb971e0 with merge ef2e2f0...

@bors
Copy link
Contributor

bors commented Sep 22, 2021

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

@bors bors merged commit ef2e2f0 into rust-lang:master Sep 22, 2021
@F3real F3real deleted the vec2 branch September 22, 2021 23:01
@F3real F3real restored the vec2 branch September 24, 2021 08:59
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.

Suggest box_string
4 participants