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: Update WebAssembly Example #5171

Merged
merged 4 commits into from
Sep 6, 2023
Merged

Rust: Update WebAssembly Example #5171

merged 4 commits into from
Sep 6, 2023

Conversation

rcoh
Copy link
Contributor

@rcoh rcoh commented Jul 27, 2023

This pull request updates the WebAssembly example to take advantage of TimeSource offered by the new version of the AWS Rust SDK & improve logging.


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

@beqqrry-aws beqqrry-aws self-assigned this Jul 27, 2023
@beqqrry-aws beqqrry-aws added Rust This issue relates to the AWS SDK for Rust (Preview) On Call Review needed This work needs an on-call review Text Review not needed This contains no text which needs review labels Jul 27, 2023
@beqqrry-aws beqqrry-aws added On Call Review complete On call review complete and removed On Call Review needed This work needs an on-call review labels Jul 27, 2023
@beqqrry-aws
Copy link
Contributor

Hey @rcoh this PR failed the Rust check.

@rlhagerm
Copy link
Contributor

@rcoh or @beqqrry-aws , do we have any path forward to resolve the failing check on this PR?

@ford-at-aws
Copy link
Contributor

@jdisanti will look into this.

@jdisanti
Copy link
Contributor

I think #5286 will resolve the issue here once it is merged. If there continues to be an issue after, I can investigate.

@cpyle0819
Copy link
Contributor

cpyle0819 commented Aug 22, 2023

@rcoh or @jdisanti please resolve conflicts.

@cpyle0819
Copy link
Contributor

@jdisanti #5286 is merged. Please re-evaluate when you can.

@Laren-AWS Laren-AWS merged commit 4803a38 into awsdocs:main Sep 6, 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.

8 participants