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

clippy::use_self #238

Open
orenbenkiki opened this issue Sep 13, 2022 · 1 comment
Open

clippy::use_self #238

orenbenkiki opened this issue Sep 13, 2022 · 1 comment

Comments

@orenbenkiki
Copy link
Contributor

The new clippy::use_self warning (in clippy::nursery) is triggered when using derive macros such as AsRefStr and/or EnumString. It seems clippy is/will become more pedantic when it comes to using Self instead of the struct name, even inside procedural macros. Right now I just disabled the warning, but I'd like to be able to turn it on in general for my own code.

@Peternator7
Copy link
Owner

iirc, we can't use Self in a number of places because earlier versions of rust didn't support Self in a number of positions historically. If you send a PR adding #[allow(..)] for the new lints, I'll get it merged and push a patch version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants