Skip to content

Conversation

@usha-mandya
Copy link
Member

Converted the tables in https://docs.docker.com/registry/storage-drivers/ from HTML to markdown format.

@usha-mandya
Copy link
Member Author

@thaJeztah Could you PTAL?

Copy link
Member

@thaJeztah thaJeztah left a 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).
Copy link
Member

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Sure, thanks.

usha-mandya and others added 3 commits February 26, 2020 13:54
Co-Authored-By: Sebastiaan van Stijn <thaJeztah@users.noreply.github.com>
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@usha-mandya usha-mandya merged commit 22da757 into docker:master Feb 27, 2020
@usha-mandya usha-mandya deleted the registry-tables-patch branch February 27, 2020 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants