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

Do not lint when imported item contains underscore #10300

Merged
merged 1 commit into from
Sep 26, 2023

Conversation

koka831
Copy link
Contributor

@koka831 koka831 commented Feb 6, 2023

fix #9942

changelog: [wildcard_imports]: No longer lints when imported items contain an item with the name _

@rustbot
Copy link
Collaborator

rustbot commented Feb 6, 2023

r? @xFrednet

(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 Feb 6, 2023
@xFrednet
Copy link
Member

xFrednet commented Feb 8, 2023

Hey, I've seen this PR, but had a pretty busy week so far. I'll review this tomorrow or on Friday, the latest! :)

@xFrednet
Copy link
Member

Hey @koka831 this is a ping from triage. There hasn't been any activity in a while, can I help you with anything? :)

@rustbot author

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties labels Mar 21, 2023
@xFrednet
Copy link
Member

I'm closing this PR due to inactivity. Hope you'll get better soon @koka831!

Anyone else reading this, feel free to continue this PR. I believe it was only one or two questions remaining.

@xFrednet xFrednet closed this May 16, 2023
@xFrednet xFrednet added S-inactive-closed Status: Closed due to inactivity and removed S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) labels May 16, 2023
@xFrednet xFrednet reopened this Sep 25, 2023
Copy link
Member

@xFrednet xFrednet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This version looks good to me. Could you please rebase on master? Then we can r+ this version. You can also create a new PR with these changes if that's easier than a rebase :)

@koka831
Copy link
Contributor Author

koka831 commented Sep 26, 2023

@xFrednet thank you for your comment. now I think I've addressed them. could you review again please?

@xFrednet
Copy link
Member

Looks good to me, thank you for coming back and updating this PR :D

@xFrednet xFrednet added S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) and removed S-inactive-closed Status: Closed due to inactivity labels Sep 26, 2023
@xFrednet
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Sep 26, 2023

📌 Commit fab9000 has been approved by xFrednet

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Sep 26, 2023

⌛ Testing commit fab9000 with merge 585b56f...

@bors
Copy link
Contributor

bors commented Sep 26, 2023

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: xFrednet
Pushing 585b56f to master...

@bors bors merged commit 585b56f into rust-lang:master Sep 26, 2023
4 checks passed
@koka831 koka831 deleted the fix/9942 branch September 26, 2023 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SIGSEGV applying clippy fixes
4 participants