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

Clippy integration is undocumented #130

Open
kornelski opened this Issue Jul 15, 2018 · 4 comments

Comments

Projects
None yet
4 participants
@kornelski
Copy link

kornelski commented Jul 15, 2018

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

This comment has been minimized.

Copy link
Contributor

Manishearth commented Jul 16, 2018

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

This comment has been minimized.

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

This comment has been minimized.

Copy link
Contributor

Manishearth commented Aug 20, 2018

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

This comment has been minimized.

Copy link

CoolOppo commented Feb 4, 2019

How do you try it with clippy?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.