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

Revamp errors in aws-smithy-eventstream #1873

Merged
merged 1 commit into from
Nov 11, 2022

Conversation

jdisanti
Copy link
Collaborator

@jdisanti jdisanti commented Oct 18, 2022

Motivation and Context

This PR revamps errors in aws-smithy-eventstream to adhere to RFC-0022: Error Context and Compatibility.

This PR is part of a series that will fully implement the RFC, tracked in #1926.

Changelog entries added in #1951.

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

@jdisanti jdisanti requested a review from a team as a code owner October 18, 2022 19:57
@github-actions
Copy link

A new generated diff is ready to view.

A new doc preview is ready to view.

@jdisanti jdisanti requested a review from a team as a code owner October 18, 2022 23:04
@github-actions
Copy link

A new generated diff is ready to view.

A new doc preview is ready to view.

@jdisanti jdisanti force-pushed the jdisanti-errors-smithy-checksums branch from 61c63d0 to 5fe1e7d Compare October 21, 2022 18:07
@jdisanti jdisanti force-pushed the jdisanti-errors-smithy-eventstream branch from c450949 to 54bcd03 Compare October 21, 2022 18:07
@jdisanti jdisanti force-pushed the jdisanti-errors-smithy-checksums branch from 5fe1e7d to 6338f4c Compare October 21, 2022 18:38
@jdisanti jdisanti force-pushed the jdisanti-errors-smithy-eventstream branch from 54bcd03 to 56a2475 Compare October 21, 2022 18:39
@github-actions
Copy link

A new generated diff is ready to view.

A new doc preview is ready to view.

@jdisanti jdisanti force-pushed the jdisanti-errors-smithy-checksums branch from 6338f4c to e053f34 Compare October 28, 2022 22:43
@jdisanti jdisanti force-pushed the jdisanti-errors-smithy-eventstream branch from 56a2475 to 3afc0db Compare October 28, 2022 22:44
@github-actions
Copy link

A new generated diff is ready to view.

A new doc preview is ready to view.

@jdisanti jdisanti marked this pull request as draft October 31, 2022 20:04
@jdisanti jdisanti force-pushed the jdisanti-errors-smithy-checksums branch from e053f34 to 55c1535 Compare November 2, 2022 20:41
@jdisanti jdisanti force-pushed the jdisanti-errors-smithy-eventstream branch from 3afc0db to aa06082 Compare November 2, 2022 20:42
@github-actions
Copy link

github-actions bot commented Nov 2, 2022

A new generated diff is ready to view.

A new doc preview is ready to view.

@jdisanti jdisanti force-pushed the jdisanti-errors-smithy-checksums branch from 55c1535 to 104d2aa Compare November 11, 2022 00:23
@jdisanti jdisanti force-pushed the jdisanti-errors-smithy-eventstream branch from aa06082 to fbb4c21 Compare November 11, 2022 00:24
@github-actions
Copy link

A new generated diff is ready to view.

A new doc preview is ready to view.

@jdisanti jdisanti force-pushed the jdisanti-errors-smithy-checksums branch from 104d2aa to 6271b8c Compare November 11, 2022 22:01
Base automatically changed from jdisanti-errors-smithy-checksums to main November 11, 2022 22:29
@jdisanti jdisanti force-pushed the jdisanti-errors-smithy-eventstream branch from fbb4c21 to 6141e74 Compare November 11, 2022 22:54
@jdisanti jdisanti marked this pull request as ready for review November 11, 2022 22:54
@jdisanti jdisanti enabled auto-merge (squash) November 11, 2022 22:54
@github-actions
Copy link

A new generated diff is ready to view.

A new doc preview is ready to view.

@jdisanti jdisanti merged commit 12f217b into main Nov 11, 2022
@jdisanti jdisanti deleted the jdisanti-errors-smithy-eventstream branch November 11, 2022 23:25
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