Skip to content
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 match_block_trailing_comma. #4145

Merged

Conversation

emilio
Copy link
Contributor

@emilio emilio commented Apr 30, 2020

Servo has used this since forever, and it'd be useful to be able to use
rustfmt stable there so that we can use the same rustfmt version in
both Firefox and Servo.

Feel free to close this if there's any reason it shouldn't be done.

Closes #3380

Servo has used this since forever, and it'd be useful to be able to use
rustfmt stable there so that we can use the same rustfmt version in
both Firefox and Servo.

Feel free to close this if there's any reason it shouldn't be done.

Closes rust-lang#3380
@calebcartwright
Copy link
Member

I think this one is probably good to stabilize as well, but want to take a final pass through the issue backlog to make sure there's no known issues.

Copy link
Member

@calebcartwright calebcartwright left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AFAICT there are no known/open issues against this config option, and it's certainly met the other stabilization criteria.

Thank you for the PR!

@emilio
Copy link
Contributor Author

emilio commented May 14, 2020

Thanks! Should this get merged then?

@calebcartwright calebcartwright merged commit 99edc88 into rust-lang:master May 16, 2020
bradleypmartin pushed a commit to bradleypmartin/rustfmt that referenced this pull request May 25, 2020
Servo has used this since forever, and it'd be useful to be able to use
rustfmt stable there so that we can use the same rustfmt version in
both Firefox and Servo.

Feel free to close this if there's any reason it shouldn't be done.

Closes rust-lang#3380
calebcartwright pushed a commit to calebcartwright/rustfmt that referenced this pull request Oct 11, 2021
Servo has used this since forever, and it'd be useful to be able to use
rustfmt stable there so that we can use the same rustfmt version in
both Firefox and Servo.

Feel free to close this if there's any reason it shouldn't be done.

Closes rust-lang#3380
calebcartwright pushed a commit to calebcartwright/rustfmt that referenced this pull request Oct 11, 2021
Servo has used this since forever, and it'd be useful to be able to use
rustfmt stable there so that we can use the same rustfmt version in
both Firefox and Servo.

Feel free to close this if there's any reason it shouldn't be done.
calebcartwright pushed a commit that referenced this pull request Oct 11, 2021
Servo has used this since forever, and it'd be useful to be able to use
rustfmt stable there so that we can use the same rustfmt version in
both Firefox and Servo.

Feel free to close this if there's any reason it shouldn't be done.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[unstable option] match_block_trailing_comma
3 participants