Skip to content

Rewrite Format to recursively format an entire SwiftSyntax tree #866

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

Merged
merged 1 commit into from
Oct 5, 2022

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Sep 30, 2022

Previously, Format required very delicate interaction with SwiftSyntaxBuilder because it did not format a syntax tree recursively. This tight coupling was difficult to understand and even harder to describe. With the new split, SwiftBasicFormat can also be used outside of SwiftSyntaxBuilder.

@ahoppen
Copy link
Member Author

ahoppen commented Sep 30, 2022

@swift-ci Please test

@ahoppen ahoppen force-pushed the ahoppen/rewrite-format branch from c19de99 to 7dec961 Compare October 5, 2022 13:37
Previously, `Format` required very delicate interaction with SwiftSyntaxBuilder because it did not format a syntax tree recursively. This tight coupling was difficult to understand and even harder to describe. With the new split, `SwiftBasicFormat` can also be used outside of SwiftSyntaxBuilder.
@ahoppen ahoppen force-pushed the ahoppen/rewrite-format branch from 7dec961 to f7c30f1 Compare October 5, 2022 13:38
@ahoppen
Copy link
Member Author

ahoppen commented Oct 5, 2022

@swift-ci Please test

@ahoppen ahoppen merged commit a31544a into swiftlang:main Oct 5, 2022
@ahoppen ahoppen deleted the ahoppen/rewrite-format branch October 5, 2022 16:28
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.

1 participant