-
Notifications
You must be signed in to change notification settings - Fork 629
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
Fix API client generation issues discovered in new model builds #708
Conversation
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.
For sts client, we see a missing dependency of strconv.
Can you add that in this PR, or can be separate; doesn't have to block this pr.
github.com/aws/aws-sdk-go-v2/service/sts [github.com/aws/aws-sdk-go-v2/service/sts.test]
./deserializers.go:1598:17: undefined: strconv
Looks like also missed transcribe streaming,
Will also need to ignore event stream operations for now until supported. |
Fixes issues with the API client codegen that were discovered in the latest set of models added to the SDK. * Fixes `bytes` package not being imported for document payload serializers. * Fixup IdempotencyToken resolver added in correct scope. If IdempotencyToken was used as a member, not on the top level shape it would cause the resolver to be generated, but the client option was never set. Regenerates API models for operation input/output (de)serialization helper function names.
Fixes issues with the API client codegen that were discovered in the latest set of models added to the SDK.
bytes
package not being imported for document payload serializers.Regenerates API models for operation input/output (de)serialization helper function names.