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

struct_field_align with shorthand and rest/base #6462

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

janos-r
Copy link

@janos-r janos-r commented Feb 5, 2025

I think I fixed it and I think it works now!
I even found a mistaken issue file where it didn't align even though it should have.

This should be able to close
#3371
#6080
#6096

This works with the new target files, but breaks in surprising
previously working places.

test with
cargo +nightly-2025-01-02 test system_tests -- --nocapture
@janos-r janos-r force-pushed the struct_field_align_with_rest branch from 6475e44 to a371652 Compare February 6, 2025 22:47
All system_tests pass now!
@janos-r janos-r force-pushed the struct_field_align_with_rest branch from 93373ca to 3b86657 Compare February 6, 2025 23:10
@janos-r janos-r marked this pull request as ready for review February 7, 2025 00:39
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.

2 participants