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

Document changeStringEnumsToEnumShapes transformer #1740

Merged
merged 2 commits into from
Apr 20, 2023

Conversation

srchase
Copy link
Contributor

@srchase srchase commented Apr 19, 2023

Adds docs for changeStringEnumsToEnumShapes transformer, including behavior from #1739

Additionally, it updates the changeTypes documentation to include its ability to transform Strings with @enum traits to Enums.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@srchase srchase requested a review from a team as a code owner April 19, 2023 16:01
@srchase srchase force-pushed the string-to-enum-transform-docs branch from 83e0118 to fcc6aff Compare April 20, 2023 16:36
@gosar
Copy link
Contributor

gosar commented Apr 20, 2023

This transform is available to 1.0 IDL too, since we load everything as 2.0 and serialize as 2.0 anyways. Should we document this in the 1.0 docs too?

@srchase srchase force-pushed the string-to-enum-transform-docs branch from 124763e to c6c5dcf Compare April 20, 2023 22:03
@srchase srchase merged commit f75cf78 into smithy-lang:main Apr 20, 2023
@srchase srchase deleted the string-to-enum-transform-docs branch April 20, 2023 22:35
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.

3 participants