-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Update EMR subcommand doc examples (applicable to v2 only) #9820
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
base: v2
Are you sure you want to change the base?
Conversation
aws-cli v2 introduced a change to how timestamps are returned in `aws emr` subcommands. Previously, a Unix time decimal was returned. Now, timestamps are returned with an ISO 8601 datetime string. This change is applicable only to aws-cli v2 docs.
aemous
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the submission.
Looks good overall, I requested a couple minor changes.
|
Thank you for the feedback and links to the style guide, I've updated the list-studios rst file appropriately. |
aemous
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made two suggestions, to make this align more with the style guide.
| @@ -0,0 +1,23 @@ | |||
| **To list available EMR Studios** | |||
|
|
|||
| The following ``list-studios`` example lists the EMR Studios in the AWS account:: | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| The following ``list-studios`` example lists the EMR Studios in the AWS account:: | |
| The following ``list-studios`` example lists the EMR Studios in the AWS account. :: |
|
|
||
| For more information, see `Monitor, delete and updatMonitor, update and delete Amazon EMR Studio resources <https://docs.aws.amazon.com/emr/latest/ManagementGuide/emr-studio-manage-studio.html>`__ in the *Amazon EMR Management Guide*. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| For more information, see `Monitor, delete and updatMonitor, update and delete Amazon EMR Studio resources <https://docs.aws.amazon.com/emr/latest/ManagementGuide/emr-studio-manage-studio.html>`__ in the *Amazon EMR Management Guide*. | |
| For more information, see `Monitor, update and delete Amazon EMR Studio resources <https://docs.aws.amazon.com/emr/latest/ManagementGuide/emr-studio-manage-studio.html>`__ in the *Amazon EMR Management Guide*. |
Issue #, if available: None
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.