Skip to content

feat: schema code generation #1599

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

Merged
merged 1 commit into from
May 29, 2025
Merged

Conversation

kuhe
Copy link
Contributor

@kuhe kuhe commented May 20, 2025

This PR adds schema code generation and an allow-list API (currently empty).


original working branch: #1592

group:
#1593 - schema types
#1594 - migrate serde functions
#1595 - schema runtime classes
#1601 - rpcv2 cbor runtime protocol
#1599 - java codegen

@kuhe kuhe requested a review from a team as a code owner May 20, 2025 19:01
@kuhe kuhe force-pushed the schema/java-codegen branch 2 times, most recently from b4454c3 to 8ce473b Compare May 20, 2025 19:49
@kuhe kuhe mentioned this pull request May 20, 2025
2 tasks
@kuhe kuhe force-pushed the schema/java-codegen branch 2 times, most recently from 52fe0ac to 76c6222 Compare May 21, 2025 19:05
@kuhe kuhe force-pushed the schema/java-codegen branch 3 times, most recently from e5141bc to ae31af9 Compare May 22, 2025 18:24
@kuhe kuhe force-pushed the schema/java-codegen branch from ae31af9 to 24c234a Compare May 29, 2025 17:45
@kuhe kuhe force-pushed the schema/java-codegen branch from 24c234a to 48f7213 Compare May 29, 2025 17:46
return SchemaTraitExtension.INSTANCE.render(trait);
}
return """
/* unhandled trait \s""" + "`" + trait.getClass().getSimpleName() + "` */";
Copy link
Contributor

Choose a reason for hiding this comment

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

would this be better returned as a warning/error of sorts?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There's going to be some additional handling in a later PR for this.

@kuhe kuhe merged commit d8255ac into smithy-lang:main May 29, 2025
10 checks passed
@kuhe kuhe deleted the schema/java-codegen branch May 29, 2025 19:22
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