-
Notifications
You must be signed in to change notification settings - Fork 8.2k
Update tables from html to markdown format #10360
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
Conversation
|
@thaJeztah Could you PTAL? |
thaJeztah
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.
left two suggestions, otherwise SGTM (tables with lots of content remain a pain-point of Markdown)
| > omit these keys to fetch temporary credentials from IAM. | ||
|
|
||
| `region`: The name of the aws region in which you would like to store objects (for example `us-east-1`). For a list of regions, you can look at http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/using-regions-availability-zones.html | ||
| `region`: The name of the aws region in which you would like to store objects (for example `us-east-1`). For a list of regions, see [Regions, Availability Zones, and Local Zones](http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/using-regions-availability-zones.html). |
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.
Looks like these descriptions overlap with the ones in the table above.
Don't think it's worth spending time on right now (as it would require checking the difference between both), but something I just noticed
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.
Sure, thanks.
Co-Authored-By: Sebastiaan van Stijn <thaJeztah@users.noreply.github.com>
….github.io into registry-tables-patch
thaJeztah
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.
LGTM, thanks!
Converted the tables in https://docs.docker.com/registry/storage-drivers/ from HTML to markdown format.