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.

Sign up for GitHub

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

Jump to bottom

Posible to install clippy for using it as a check? #840

Closed
hoijui opened this issue Dec 7, 2023 · 4 comments · Fixed by #843
Closed

Posible to install clippy for using it as a check? #840

hoijui opened this issue Dec 7, 2023 · 4 comments · Fixed by #843
Labels
C-enhancement
Milestone
v0.16.0

Comments

@hoijui
Copy link

hoijui commented Dec 7, 2023

I want to use cargo clippy as the check, so I would need to be able to specify that rustup should also install clippy for each rust version to be checked. Is this possible? does this make sense?
@hoijui
Copy link
Author

hoijui commented Dec 7, 2023

Is this possibly related to #701?

@foresterre foresterre added this to the v0.16.0 milestone Dec 10, 2023
@foresterre
Copy link
Owner

Not possible out of the box right now, but should certainly be possible. Started working on this now.

@foresterre foresterre mentioned this issue Dec 10, 2023
Merged
@foresterre
Copy link
Owner

Should now be possible, with cargo install cargo-msrv --version 0.16.0-beta.18, and cargo msrv (...) --add-component clippy -- cargo clippy

@hoijui
Copy link
Author

hoijui commented Dec 10, 2023

woow.. thank you @foresterre ! :-)
that was quick. have a good evening and week!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-enhancement
Projects
None yet
Milestone
v0.16.0
Development

Successfully merging a pull request may close this issue.

feat: Add option to add components with rustup foresterre/cargo-msrv
2 participants
@hoijui @foresterre