-
Notifications
You must be signed in to change notification settings - Fork 898
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
Include rustfmt and clippy components by default #1569
Comments
I'm currently implementing 'profiles' for Rustup which will include tools, the default profile will include Rustfmt and Clippy. |
Profiles were released with 1.20, this is "done" |
@Stargateur the README for Rustup clearly states the content of the |
@kinnison Maybe I miss understand, the issue is about docker, so my question was where this information is on docker readme, I'm using "rust:latest" as image and I would like to know if this include fmt, that seem to not be the case. My question more simply is there a official docker image with rustfmt installed ? |
Oh I'm sorry, I didn't realise -- |
Originated in rust-lang/docker-rust#37
rustfmt
andclippy
components are widely used tools by many projects. However, there is some friction with using them on CI and dockerized environments:rustup add component rustfmt
any time we're using Docker image for that purpose. On CIs it just unnecessarily increases network activity. In CLI usage it becomes trickier asdocker run --rm -v "$(pwd)":/app -w /app rust:1.31 cargo fmt
just fails.rustfmt
/clippy
on nightly.The motivation of official Rust Docker images is to follow upstream tools as mush as possible without providing difference with official installations. As the result, these issues make people to workaround/hack or come up with their own Docker images.
Can we somehow provide a smoother experience?
The text was updated successfully, but these errors were encountered: