Skip to content
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

Bring over structured Swift syntax from OpenAPI #1722

Merged
merged 2 commits into from
Nov 27, 2023

Conversation

stefanadranca
Copy link
Collaborator

Motivation:

We’ll need to be able to represent the code to be generated as structured Swift types for the new Codegen library. This is effectively what swift-syntax provides but we can’t (at time of writing) take it as a dependency because of its unstable API. Swift OpenAPI Generator has already provided a number of these types as its StructuedSwiftRepresentation.

Modifications:

  • Copied StructuredSwiftRepresentation.swift inside the Codegen library module
  • Mentioned the usage in NOTICES.txt

Result:

Codegen library will have a representation of the code to generate as structured Swift types.

Motivation:

We’ll need to be able to represent code to generate as structured Swift types for the new Codegen library.
This is effectively what swift-syntax provides but we can’t (at time of writing) take it as a dependency
because of its unstable API. Swift OpenAPI Generator has already provided a number of these types as its StructuedSwiftRepresentation.

Modifications:

- Copied StructuredSwiftRepresentation.swift inside the Codegen library module
- Mentioned the usage in NOTICES.txt

Result:

Codegen library will have a representation of the code to generate as structured Swift types.
@glbrntt glbrntt added the version/v2 Relates to v2 label Nov 27, 2023
@glbrntt glbrntt merged commit 688287c into grpc:main Nov 27, 2023
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
version/v2 Relates to v2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants