Skip to content

Updating API Changelog to add /distribution/{name}/json endpoint#33147

Merged
thaJeztah merged 1 commit intomoby:masterfrom
nishanttotla:docs-update-distribution-json-endpoint
May 11, 2017
Merged

Updating API Changelog to add /distribution/{name}/json endpoint#33147
thaJeztah merged 1 commit intomoby:masterfrom
nishanttotla:docs-update-distribution-json-endpoint

Conversation

@nishanttotla
Copy link
Contributor

This is a quick follow up for #32061.

@nishanttotla
Copy link
Contributor Author

cc @mstanleyjones @thaJeztah let me know if you prefer different language in the docs here.

Copy link
Contributor

Choose a reason for hiding this comment

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

Will this work with an image ID too/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Image ID can be passed, but it'll just return an error, because it's not possible to reach the registry with an Image ID.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think service inspect should have back ticks. But I also suspect you didn't mean to change this and your editor removed some trailing whitespace.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, yes you're right my editor removed the whitespace. Should I add back ticks in this PR?

Signed-off-by: Nishant Totla <[email protected]>
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah force-pushed the docs-update-distribution-json-endpoint branch from d77b149 to a89dd03 Compare May 11, 2017 10:15
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.

rebased, and removed the unrelated whitespace change

LGTM

@thaJeztah thaJeztah merged commit 1f55b71 into moby:master May 11, 2017
@GordonTheTurtle GordonTheTurtle added this to the 17.06.0 milestone May 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants