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

Update documentation to include changes in aws jslib v0.9.0 #1276

Merged
merged 13 commits into from
Aug 15, 2023
Merged

Conversation

oleiade
Copy link
Member

@oleiade oleiade commented Aug 2, 2023

This Pull Request adapts the aws jslib documentation section to fit the changes introduced by version 0.9.0.

This big release makes almost all the APIs facing the user asynchronous and compatible with the async/await syntax. Previous releases exposing synchronous codes will stay out there, untouched. Still, as the documentation website is not versioned, only the documentation for the async version will be available once this PR is merged.

This is a massive PR, I apologize for that, but I still look forward for you folks spotting all the tiny mistakes and "I forgot" occurrences probably scattered across the PR 🙇🏻

@oleiade oleiade self-assigned this Aug 2, 2023
@oleiade oleiade added the Area:extensions Issues about the extension ecosystem label Aug 2, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Aug 2, 2023

There's a version of the docs published here:

https://mdr-ci.staging.k6.io/docs/refs/pull/1276/merge

It will be deleted automatically in 30 days.

Copy link
Contributor

@immavalls immavalls left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, lots of work to change all the examples 💪🏼. I tested most examples in the main pages (AwsConfig, S3Client, KMSClient, SecretsManagerClient, and SignatureV4), and they work very well.
Unrelated, are we still missing the Kinesis docs?

Copy link
Collaborator

@heitortsergent heitortsergent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left a few comments, mostly suggestions 😊. Thanks for updating all those examples, that must have taken quite some time!

…ckquote.view.js

Co-authored-by: Heitor Tashiro Sergent <[email protected]>
@oleiade oleiade merged commit 25052b9 into main Aug 15, 2023
@oleiade oleiade deleted the aws/v0.9.0 branch August 15, 2023 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area:extensions Issues about the extension ecosystem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants