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

Rust: Fix example compilation #5286

Merged
merged 3 commits into from
Aug 22, 2023
Merged

Rust: Fix example compilation #5286

merged 3 commits into from
Aug 22, 2023

Conversation

jdisanti
Copy link
Contributor

There are some breaking changes in the latest Rust SDK release that require changes to examples in order to fix compile errors. This PR fixes all the Rust examples with the latest SDK.

The dependencies need to get updated to use aws-sdk-rust/next in some cases where code changes to the SDK were required to fix the examples. The examples need to be fixed in aws-doc-sdk-examples before we can release those SDK changes since the SDK release process requires successfully compiling examples (or at least, now it does after we fixed a bug where it wasn't).


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

@github-actions github-actions bot added the Rust This issue relates to the AWS SDK for Rust (Preview) label Aug 18, 2023
@beqqrry-aws beqqrry-aws added On Call Review needed This work needs an on-call review Text Review not needed This contains no text which needs review labels Aug 21, 2023
@ford-at-aws ford-at-aws added On Call Review complete On call review complete and removed On Call Review needed This work needs an on-call review labels Aug 21, 2023
@cpyle0819 cpyle0819 merged commit 577a559 into awsdocs:main Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
On Call Review complete On call review complete Rust This issue relates to the AWS SDK for Rust (Preview) Text Review not needed This contains no text which needs review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants