Skip to content

Use the grouped diagnostics formatter in swift-parser-cli print-diags #1391

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
Mar 14, 2023

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Mar 8, 2023

The grouped diagnostics formatter is what we use to render diagnsotics from the compiler, so I think that swift-parser-cli print-diags should match that formatting style.

@ahoppen ahoppen requested a review from DougGregor March 8, 2023 16:52
@ahoppen
Copy link
Member Author

ahoppen commented Mar 8, 2023

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented Mar 8, 2023

@swift-ci Please test

@DougGregor
Copy link
Member

Looks like it needs formatting and a rerun.

The grouped diagnostics formatter is what we use to render diagnsotics from the compiler, so I think that `swift-parser-cli print-diags` should match that formatting style.
@ahoppen ahoppen force-pushed the ahoppen/grouped-formatter branch from 2181208 to a4cfdd8 Compare March 10, 2023 18:14
@ahoppen
Copy link
Member Author

ahoppen commented Mar 10, 2023

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented Mar 14, 2023

@swift-ci Please test macOS

@ahoppen ahoppen merged commit 20d5fb9 into swiftlang:main Mar 14, 2023
@ahoppen ahoppen deleted the ahoppen/grouped-formatter branch March 14, 2023 16:45
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.

2 participants