-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
Update rustfmt #56652
Update rustfmt #56652
Conversation
This doesn’t pass tidy because |
(rust_highfive has picked a reviewer for you, use r? to override) |
|
We do override rustfmt with the in-tree copy here but RLS pulls in older We'd need to land rust-lang/rls#1178 (bumps cc @nrc |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
With rust-lang/rls#1178 merged we can also update RLS in-tree to rust-lang/rls@8c264cc. |
Update Rustfmt and RLS Supersedes #56652, hopefully fixes toolstate. r? @SimonSapin
☔ The latest upstream changes (presumably #56666) made this pull request unmergeable. Please resolve the merge conflicts. |
Superseded by #56666 |
Pick up rust-lang/rustfmt#3236