Skip to content

Move swiftsyntax/Misc gyb to syntax generator#1128

Merged
kimdv merged 1 commit intoswiftlang:mainfrom
kimdv:kimdv/add-codgen-for-swiftsyntax
Dec 13, 2022
Merged

Move swiftsyntax/Misc gyb to syntax generator#1128
kimdv merged 1 commit intoswiftlang:mainfrom
kimdv:kimdv/add-codgen-for-swiftsyntax

Conversation

@kimdv
Copy link
Copy Markdown
Contributor

@kimdv kimdv commented Dec 10, 2022

No description provided.

@kimdv kimdv requested a review from ahoppen as a code owner December 10, 2022 16:14
@kimdv kimdv changed the title Move Misc gyb to syntax generator Move swiftsyntax/Misc gyb to syntax generator Dec 10, 2022
@kimdv kimdv force-pushed the kimdv/add-codgen-for-swiftsyntax branch from f35abdc to 1513a85 Compare December 10, 2022 16:25
@kimdv
Copy link
Copy Markdown
Contributor Author

kimdv commented Dec 10, 2022

@swift-ci please test

@kimdv kimdv force-pushed the kimdv/add-codgen-for-swiftsyntax branch from 1513a85 to 51637d1 Compare December 12, 2022 07:17
@kimdv kimdv mentioned this pull request Dec 12, 2022
3 tasks
.executable(name: "generate-swiftbasicformat", targets: ["generate-swiftbasicformat"]),
.executable(name: "generate-swiftideutils", targets: ["generate-swiftideutils"]),
.executable(name: "generate-swiftparser", targets: ["generate-swiftparser"]),
.executable(name: "generate-swiftsyntax", targets: ["generate-swiftsyntax"]),
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not part of this PR but what do you think about having a single executable and adding an option that allows filtering of the files that should be generated?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It sounds like a good idea, as this not scales that well 😅

I will come with a suggestion when this is merged

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you so much!

@kimdv kimdv force-pushed the kimdv/add-codgen-for-swiftsyntax branch 2 times, most recently from b75f780 to 881fd3e Compare December 13, 2022 10:52
Copy link
Copy Markdown
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

@kimdv
Copy link
Copy Markdown
Contributor Author

kimdv commented Dec 13, 2022

@swift-ci please test

@kimdv kimdv force-pushed the kimdv/add-codgen-for-swiftsyntax branch from 881fd3e to fa04041 Compare December 13, 2022 11:21
@kimdv
Copy link
Copy Markdown
Contributor Author

kimdv commented Dec 13, 2022

@swift-ci please test

@kimdv kimdv merged commit eb117dc into swiftlang:main Dec 13, 2022
@kimdv kimdv deleted the kimdv/add-codgen-for-swiftsyntax branch December 14, 2022 14:24
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