llogiq

Results 228 comments of llogiq

Thanks for seeing this through. @bors r+

Ok, let's try once more. @bors r+

Agreed, let's not complicate the solution. As long as both lints don't step on each other's toes by issuing incompatible suggestions, having two lints that don't care about each other...

Please keep in mind that we deem false positives a worse outcome than false negatives. I'd be OK with having either a config that makes the lint apply everywhere instead...

I would fully support that argument for a `suspicious` lint, but not for a `style` lint. Please note that a lint rarely works in isolation – clippy usually runs many...

Ok, that's the minimal solution. I had a list of paths in mind that should be treated as format!-like macros, possibly with `*` wildcards, which would give users full flexibility.

The base idea was to have fully qualified paths and then perhaps extend that to allow for wild cards. But I agree, doing the minimal thing first is entirely acceptable.

@Alexendoo I'd also be OK with merging this now and extending in a follow up PR. What do you think?