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

Revert "2024: Assignment operator RHS indentation" #343

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Dec 9, 2024

Reverts #341. This isn't implemented. If and when it is ready, this can be added back in.

Tracking:

@traviscross
Copy link
Contributor

cc @calebcartwright @ytmimi

Do we think this is going to happen in time for the edition?

@ytmimi
Copy link

ytmimi commented Dec 10, 2024

I believe so. We've got rust-lang/rustfmt#6305 already open, and with the scope of this reduced to just assignments I think that PR is just about ready to go. I'll review it again this week.

@calebcartwright
Copy link
Member

I believe so. We've got rust-lang/rustfmt#6305 already open, and with the scope of this reduced to just assignments I think that PR is just about ready to go. I'll review it again this week.

i've also got a different version of this that's implemented and ready, just need to get the pr up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants