Skip to content

Move gyb files #1158

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

Closed
wants to merge 8 commits into from
Closed

Move gyb files #1158

wants to merge 8 commits into from

Conversation

kimdv
Copy link
Contributor

@kimdv kimdv commented Dec 19, 2022

Yet another PR moving gyb files to swift code gen ☠️

I'm not able to move the Trivia.swift.gyb because of Swift/Character.swift:177: Fatal error: Can't form a Character from a String containing more than one extended grapheme cluster
Even we have it in raw strings using # it's not working.

@kimdv kimdv requested a review from ahoppen as a code owner December 19, 2022 08:02
@kimdv kimdv force-pushed the kimdv/move-gyb-files branch from 064f1d9 to eb875f4 Compare December 19, 2022 08:03
@kimdv
Copy link
Contributor Author

kimdv commented Dec 19, 2022

@swift-ci please test

@kimdv kimdv force-pushed the kimdv/move-gyb-files branch 2 times, most recently from 7079d45 to 873e9f4 Compare December 19, 2022 14:04
@kimdv kimdv force-pushed the kimdv/move-gyb-files branch from 3882177 to db80ba9 Compare December 25, 2022 14:52
@ahoppen
Copy link
Member

ahoppen commented Dec 27, 2022

Thanks for killing the remaining gyb.

When the dependencies are merged, could you open separate PRs for the individual files? That makes reviewing them easier.

You can also open a PR for Character.swift and I’ll try to figure out what’s going wrong.

@kimdv
Copy link
Contributor Author

kimdv commented Jan 8, 2023

Closing in favour of individual PRs

@kimdv kimdv closed this Jan 8, 2023
@kimdv kimdv deleted the kimdv/move-gyb-files branch January 9, 2023 07:58
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