-
Couldn't load subscription status.
- Fork 961
support formatting half open ranges #4044
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
support formatting half open ranges #4044
Conversation
123fc01 to
27f2120
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the PR! LGTM with the comment resolved.
14da660 to
8c43b43
Compare
8c43b43 to
dbf385a
Compare
| context, | ||
| ) { | ||
| let missing_comments = match rewrite_missing_comment(comments_span, cond_shape, context) | ||
| { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, let's keep master green 🍏
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
|
backported in #4105 |
Adds support for formatting half open range syntax (
X..,..X, and..=X).In #4022 the rustc-ap-* crates were upgraded so that rustfmt will no longer throw a parser error on the half open range syntax, but this PR actually supports formatting those ranges.
Resolves #4009
Refs rust-lang/rust#67264
Refs rust-lang/rust#67258
Refs rust-lang/rust#68120