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

Don't lint manual_non_exhaustive when the enum variant is used #8645

Merged
merged 2 commits into from
Apr 12, 2022

Conversation

Jarcho
Copy link
Contributor

@Jarcho Jarcho commented Apr 6, 2022

fixes #5714

changelog: Don't lint manual_non_exhaustive when the enum variant is used

@rust-highfive
Copy link

r? @xFrednet

(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 Apr 6, 2022
@Jarcho Jarcho force-pushed the manual_non_exhaustive_5714 branch from 5e84122 to b0f8a31 Compare April 6, 2022 14:12
@xFrednet
Copy link
Member

xFrednet commented Apr 11, 2022

This looks good to me, thank you for the implementation!

I've marked one question, if this change was intentional, you can r=me (bors r=xFrednet) this PR 🙃

@bors d+

@bors
Copy link
Collaborator

bors commented Apr 11, 2022

📌 Commit b0f8a31 has been approved by xFrednet)`

@bors
Copy link
Collaborator

bors commented Apr 11, 2022

⌛ Testing commit b0f8a31 with merge 2cd9e98...

bors added a commit that referenced this pull request Apr 11, 2022
Don't lint `manual_non_exhaustive` when the enum variant is used

fixes #5714

changelog: Don't lint `manual_non_exhaustive` when the enum variant is used
@xFrednet
Copy link
Member

xFrednet commented Apr 11, 2022

It seems like bors was a bit too quick and took my inline code mention as the real one 😅

@bors r-

@xFrednet
Copy link
Member

Looks like I messed up with some commands. Let's try this again:

@bors delegate+

@bors
Copy link
Collaborator

bors commented Apr 11, 2022

✌️ @Jarcho can now approve this pull request

@rust-lang rust-lang deleted a comment from bors Apr 11, 2022
@Jarcho
Copy link
Contributor Author

Jarcho commented Apr 12, 2022

@bors r+

@bors
Copy link
Collaborator

bors commented Apr 12, 2022

📌 Commit b0f8a31 has been approved by Jarcho

@bors
Copy link
Collaborator

bors commented Apr 12, 2022

⌛ Testing commit b0f8a31 with merge d8c97e6...

@bors
Copy link
Collaborator

bors commented Apr 12, 2022

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

@bors bors merged commit d8c97e6 into rust-lang:master Apr 12, 2022
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.

manual_non_exhaustive should not warn on a used enum variant
4 participants