-
Notifications
You must be signed in to change notification settings - Fork 95
feat: Add codegen build plugin to AWS services #2053
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
base: epic/sbs
Are you sure you want to change the base?
Conversation
| static var SmithyTimestamps: Self { .product(name: "SmithyTimestamps", package: "smithy-swift") } | ||
| static var SmithyWaitersAPI: Self { .product(name: "SmithyWaitersAPI", package: "smithy-swift") } | ||
| static var SmithyXML: Self { .product(name: "SmithyXML", package: "smithy-swift") } | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything below is just a copy of the changes in Package.base.txt above
| rm -rf Sources/Services/* | ||
| rm -rf Tests/Services/* | ||
| rm -rf SmokeTests/* | ||
| rm -rf Sources/Core/AWSIdentity/InternalClients/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete internal clients so they are regenerated completely during codegen
| // Smithy plugins | ||
| static var SmithyCodeGenerator: Self { .plugin(name: "SmithyCodeGenerator", package: "smithy-swift") } | ||
| } | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Above, a convenience var with the code generator plugin is defined.
Below, the plugin is inserted into all of our service clients (public and internal.)
Description of changes
Applies the Swift-native codegen plugin defined in smithy-lang/smithy-swift#993 to AWS service clients, including the internal ones.
New/existing dependencies impact assessment, if applicable
No new dependencies were added to this change.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.