Skip to content

feat(clients): support awsQueryError trait#3998

Merged
trivikr merged 7 commits intoaws:mainfrom
trivikr:aws-query-error-trait
Sep 29, 2022
Merged

feat(clients): support awsQueryError trait#3998
trivikr merged 7 commits intoaws:mainfrom
trivikr:aws-query-error-trait

Conversation

@trivikr
Copy link
Member

@trivikr trivikr commented Sep 28, 2022

Issue

Internal JS-2681

Description

Adds support for awsQueryError trait

Testing

  • CI is successful
  • Integration tests are successful

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@trivikr trivikr changed the title feat: support awsQueryError trait feat(clients): support awsQueryError trait Sep 28, 2022
@trivikr
Copy link
Member Author

trivikr commented Sep 28, 2022

The Java CI is failing, as the change from smithy-typescript in smithy-lang/smithy-typescript#598 is not available. Verified that the CI commands from codegen-ci are successful in the workspace.

@trivikr trivikr force-pushed the aws-query-error-trait branch from b51cd36 to 2926a46 Compare September 29, 2022 16:27
@trivikr trivikr marked this pull request as ready for review September 29, 2022 16:27
@trivikr trivikr requested a review from a team as a code owner September 29, 2022 16:27
@trivikr trivikr merged commit 384cbd7 into aws:main Sep 29, 2022
@trivikr trivikr deleted the aws-query-error-trait branch September 29, 2022 17:41
siddsriv pushed a commit to siddsriv/aws-sdk-js-v3 that referenced this pull request Oct 3, 2022
@github-actions
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments