bat icon indicating copy to clipboard operation
bat copied to clipboard

Apply some `clippy::pedantic` suggestions

Open acuteenvy opened this issue 5 months ago • 3 comments

Thank you. Let's merge #2755 (and possibly others) first in order not to create too many conflicts.

sharkdp avatar Jan 17 '24 21:01 sharkdp

#2755 has been merged now.

sharkdp avatar Jan 21 '24 19:01 sharkdp

It might sound silly, but can you split this out into separate PRs with simple-to-review and uncontroversial changes please? Right now the PR is a bit too big for my taste I'm afraid.

Enselic avatar Jan 29 '24 16:01 Enselic

Sure, each clippy lint in a separate PR?

acuteenvy avatar Feb 16 '24 18:02 acuteenvy

The smaller the better as far as I'm concerned (but not too small of course)

Enselic avatar Feb 16 '24 18:02 Enselic

Let's close this for now.

sharkdp avatar Feb 23 '24 20:02 sharkdp