-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Stabilize Formatter alignment #51078
Conversation
r? @cramertj (rust_highfive has picked a reviewer for you, use r? to override) |
96a02c5
to
154f0cc
Compare
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 |
r=me with the attribute fixed |
☔ The latest upstream changes (presumably #51082) made this pull request unmergeable. Please resolve the merge conflicts. |
3bda931
to
09608fc
Compare
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 |
09608fc
to
fb447f1
Compare
@bors: r=SimonSapin |
📌 Commit fb447f1 has been approved by |
Stabilize Formatter alignment Fixes #27726. cc @SimonSapin
☀️ Test successful - status-appveyor, status-travis |
Shouldn't core::fmt::Alignment be re-exported as std::fmt::Alignment? |
That's a good point. I'll do it as well. |
…Sapin Reexport fmt::Alignment into std Follow-up of #51078.
Fixes #27726.
cc @SimonSapin