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

fix single_component_path_imports FP on self::<import>::.. #10566

Merged
merged 1 commit into from
Apr 7, 2023

Conversation

ebobrow
Copy link
Contributor

@ebobrow ebobrow commented Mar 30, 2023

fixes #10549

I noticed that a couple functions in the file I was working on took cx as a parameter but didn't use them, so I removed that. Can revert if desired because it isn't related to my changes.

changelog: [single_component_path_imports] don't suggest removing import when it is used as self::<import>::..

@rustbot
Copy link
Collaborator

rustbot commented Mar 30, 2023

r? @giraffate

(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 Mar 30, 2023
@giraffate
Copy link
Contributor

@bors r+

Thanks!

@bors
Copy link
Collaborator

bors commented Apr 7, 2023

📌 Commit a216553 has been approved by giraffate

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Apr 7, 2023

⌛ Testing commit a216553 with merge 9408d01...

@bors
Copy link
Collaborator

bors commented Apr 7, 2023

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: giraffate
Pushing 9408d01 to master...

1 similar comment
@bors
Copy link
Collaborator

bors commented Apr 7, 2023

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: giraffate
Pushing 9408d01 to master...

@bors bors merged commit 9408d01 into rust-lang:master Apr 7, 2023
@ebobrow ebobrow deleted the iss10549 branch April 7, 2023 00:30
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.

FP single-component-path-imports : self::<import>::...
4 participants