-
Notifications
You must be signed in to change notification settings - Fork 440
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
Move raw syntax validation from gyb to code gen #1332
Conversation
e7c470c
to
1a6a924
Compare
There was a problem hiding this 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.
@swift-ci Please test |
Haven't run the code gen yet. |
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. |
1a6a924
to
b8770fb
Compare
b8770fb
to
f46d629
Compare
ef78caa
to
d1f39ec
Compare
@swift-ci please test |
d1f39ec
to
9a0941b
Compare
@swift-ci please test |
I think it's ready now @ahoppen 🥳 |
@swift-ci Please test macOS |
There was a problem hiding this 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.
The last gym file 🥳
The removal of gym files will come in a follow up PR when this is merged