Skip to content

Move raw syntax validation from gyb to code gen #1332

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

Conversation

kimdv
Copy link
Contributor

@kimdv kimdv commented Feb 9, 2023

The last gym file 🥳

The removal of gym files will come in a follow up PR when this is merged

@kimdv kimdv requested a review from ahoppen as a code owner February 9, 2023 14:01
@kimdv kimdv force-pushed the kimdv/move-raw-syntax-validation-from-gyb-to-code-gen branch from e7c470c to 1a6a924 Compare February 11, 2023 17:54
Copy link
Member

@ahoppen ahoppen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. This looks good to me.

@ahoppen
Copy link
Member

ahoppen commented Feb 13, 2023

@swift-ci Please test

@kimdv
Copy link
Contributor Author

kimdv commented Feb 13, 2023

Haven't run the code gen yet.
I'm still not able to do multi string literals

@ahoppen
Copy link
Member

ahoppen commented Feb 13, 2023

Ah, I thought you did now because the generated file changed. Let me know when the multi-line string literal issue is fixed and this PR is ready for review.

@kimdv kimdv force-pushed the kimdv/move-raw-syntax-validation-from-gyb-to-code-gen branch from 1a6a924 to b8770fb Compare February 15, 2023 12:54
@kimdv kimdv force-pushed the kimdv/move-raw-syntax-validation-from-gyb-to-code-gen branch from b8770fb to f46d629 Compare February 24, 2023 14:54
@kimdv kimdv force-pushed the kimdv/move-raw-syntax-validation-from-gyb-to-code-gen branch 2 times, most recently from ef78caa to d1f39ec Compare February 26, 2023 15:26
@kimdv
Copy link
Contributor Author

kimdv commented Feb 26, 2023

@swift-ci please test

@kimdv kimdv requested a review from ahoppen February 26, 2023 15:50
@kimdv kimdv force-pushed the kimdv/move-raw-syntax-validation-from-gyb-to-code-gen branch from d1f39ec to 9a0941b Compare February 26, 2023 19:34
@kimdv
Copy link
Contributor Author

kimdv commented Feb 26, 2023

@swift-ci please test

@kimdv
Copy link
Contributor Author

kimdv commented Feb 27, 2023

I think it's ready now @ahoppen 🥳

@ahoppen
Copy link
Member

ahoppen commented Feb 27, 2023

@swift-ci Please test macOS

Copy link
Member

@ahoppen ahoppen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you. This looks good to me. Just a few comments inline that I noticed but that aren’t really part of this PR.

@kimdv kimdv merged commit f248906 into swiftlang:main Feb 27, 2023
@kimdv kimdv deleted the kimdv/move-raw-syntax-validation-from-gyb-to-code-gen branch February 27, 2023 16:16
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