Add Rustfmt by nrc · Pull Request #44533 · rust-lang/rust (original) (raw)

Do you think we should wait for #43628 to have a TOML file to enable/disable the tool?

We're already building with the RLS, so it doesn't feel like we need to wait for that (I can't imagine a change to the compiler causing a Rustfmt test to fail, just stop it building), and landing this will make it easier to land changes which are breaking.

Do you want to go ahead and create a rustfmt-preview component for rustup? Or leave that to a future PR?

I was going to do this later, there are probably a few things we should do in rustfmt before it should be rustup-installable.

Will address the inline points...