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

rustfmt: revert overflow_delimited_expr=true default in style_edition=2024 #136317

Conversation

ytmimi
Copy link
Contributor

@ytmimi ytmimi commented Jan 30, 2025

Closes #136224

There are still a few issues with overflow_delimited_expr where the option produces strange formatting. overflow_delimited_expr is still an unstable rustfmt configuration option, and after some discussion the consensus was to revert the overflow_delimited_expr change for style_edition 2024.

Zulip chat

…ion=2024`

There are still a few issues with `overflow_delimited_expr` where the
option produces strange formatting. `overflow_delimited_expr` is still
an unstable rustfmt configuration option, and after some discussion the
consensus was to revert the `overflow_delimited_expr` change for
style_edition 2024.
@rustbot
Copy link
Collaborator

rustbot commented Jan 30, 2025

r? @Mark-Simulacrum

rustbot has assigned @Mark-Simulacrum.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 30, 2025
@rustbot
Copy link
Collaborator

rustbot commented Jan 30, 2025

Some changes occurred in src/tools/rustfmt

cc @rust-lang/rustfmt

@ytmimi
Copy link
Contributor Author

ytmimi commented Jan 30, 2025

Closing in favor of #136312

@ytmimi ytmimi closed this Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Style edition 2024: overflow_delimited_expr behavior should be reconsidered before stabilization
3 participants