-
Notifications
You must be signed in to change notification settings - Fork 13.5k
Add a missing fmt::Debug impl lint #21610
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
Conversation
Thanks! I think it's a bit too strong to set the default level to warn for this lint though. We've seen real compile time hits as well as code blow up sizes in the past from just adding From a library author's perspective it's unfortunate to accidentally leave this out, but it's a backwards compatible change to add one at any time. |
I was under the impression that I disabled the lint for all of the main rustc crates because I didn't feel like adding hundreds of Debug implementations in this PR. I don't see any reason to add a ton of impls in the |
To me that guideline states that all public types should have a |
Updated to be allow by default. We may want to set the missing_copy_implementations to default allow as well. |
I would love to do so for that! It may want to have a separate PR though to prevent entangling the two. |
Yeah, for sure. |
@bors: r+ a7ad08c |
Closes #20855
r? @alexcrichton