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

All clippy warnings that currently block compliation #235

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

koshell
Copy link

@koshell koshell commented Jun 18, 2024

Versions:
rustc 1.79.0
clippy 0.1.79

Warnings:
clippy::default_constructed_unit_structs
clippy::single_char_pattern
clippy::len_zero
clippy::clone_on_copy
clippy::duplicated_attributes

After going back through to check for these I realised all of them, except clippy::duplicated_attributes, occur in tests and therefore can likely be ignored.

Changing

#![deny(warnings)]

to

#![cfg_attr(not(test), deny(warnings))]

would allow the crate to compile (assuming clippy::duplicated_attributes is resolved).

However I would recommend reading over this and considering if you even want to keep deny(warnings) in general.

@antifuchs
Copy link
Collaborator

antifuchs commented Jun 18, 2024

Hrm, it seems that I was building this with the wrong clippy version all along, can now repro them. I think your suggestion of turning the deny(warnings) off in tests is sound - it's not like we care much about perf or other little nits there.

duplicated_attributes is worth fixing though, of course (:

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

Successfully merging this pull request may close these issues.

2 participants