Skip to content

Conversation

dtolnay
Copy link
Member

@dtolnay dtolnay commented Mar 27, 2021

The false positive in #6984 (which started appearing in the most recent nightly) makes this lint actively damaging for enums such as https://github.com/dtolnay/syn/blob/17332d952a3bc58e95692f313d595bd019f31d39/src/expr.rs#L227-L251, so I believe it is worth turning off for now until fixed, even for people who lint their code with warn(clippy::pedantic).

changelog: Bump match_wildcard_for_single_variants lint from pedantic to nursery

@rust-highfive
Copy link

r? @giraffate

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

Thanks!

The false positive may be fixed by #6988.

@giraffate
Copy link
Contributor

@dtolnay #6988 is merged, so #6984 will be fixed. Could you still need this PR?

@bors
Copy link
Contributor

bors commented Apr 6, 2021

☔ The latest upstream changes (presumably #6931) made this pull request unmergeable. Please resolve the merge conflicts.

Copy link
Member Author

@dtolnay dtolnay left a comment

Choose a reason for hiding this comment

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

Confirmed fixed in nightly-2021-04-09, so this is no longer needed.

@dtolnay dtolnay closed this Apr 9, 2021
@dtolnay dtolnay deleted the matchprivate branch April 9, 2021 02:27
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.

4 participants