Skip to content

Don't lint std_instead_of_core on std::env #9243

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jul 25, 2022
Merged

Conversation

Jarcho
Copy link
Contributor

@Jarcho Jarcho commented Jul 25, 2022

fixes #9239

This also reorders the execution of the lint to do as little as possible in the case where the path doesn't start with std or alloc.

changelog: std_instead_of_core: Don't lint on use std::env
changelog: std_instead_of_alloc: Don't lint use std::vec twice

@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 Jul 25, 2022
* Don't call `TyCtxt::crate_name` unless necessary
* Don't lint on `use std::env`
* Only lint once on `use std::vec`
@c410-f3r
Copy link
Contributor

@Jarcho Thanks

@Manishearth
Copy link
Member

@bors r+

oops, shoulda caught that in the original PR

@bors
Copy link
Contributor

bors commented Jul 25, 2022

📌 Commit 6bc024d has been approved by Manishearth

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Jul 25, 2022

⌛ Testing commit 6bc024d with merge c76db5c...

@bors
Copy link
Contributor

bors commented Jul 25, 2022

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

# for free to join this conversation on GitHub. Already have an account? # 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.

Plain wrong std_instead_of_core suggestion
5 participants