Skip to content
This repository has been archived by the owner on Nov 24, 2023. It is now read-only.

Clippy integration is undocumented #130

Closed
kornelski opened this issue Jul 15, 2018 · 8 comments
Closed

Clippy integration is undocumented #130

kornelski opened this issue Jul 15, 2018 · 8 comments

Comments

@kornelski
Copy link
Contributor

Clippy is mentioned in the README, but the README contains no information on how to use cargo-fix and clippy together.

cargo fix --help does not contain information about Clippy integration either.

In the Clippy's README and help there is nothing about rustfix either.

@Manishearth
Copy link
Member

I think right now the focus is for this to be used for editions, and post-edition worry about clippy (the integration then will be different too, since clippy is changing a bunch)

@estk
Copy link

estk commented Aug 20, 2018

@Manishearth, totally understandable. However it isn't clear if this works with Clippy or not. At the least it would be helpful to know that it does not. I'd be happy to update the docs if you could provide an outline here of what the status is.

@Manishearth
Copy link
Member

It does work with clippy, it's just not polished right now. It's a complicated answer -- you can get it to work with clippy, but it won't work well, and it's not how it will eventually work.

@CoolOppo
Copy link

CoolOppo commented Feb 4, 2019

How do you try it with clippy?

@luciusmagn
Copy link

there's a clippy integration? Neat! How do I try it?

@Manishearth
Copy link
Member

You're not supposed to use it yet. We're working on properly exposing it, it's a part of rust-lang/rust-clippy#3837

@luciusmagn
Copy link

Alrighty 😄 thanks a lot for the lightning fast response! Is there anything I can do to help?

@Manishearth
Copy link
Member

Not at the moment, this is mostly blocked on review from cargo.

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

No branches or pull requests

5 participants