Skip to content

manifest: use descriptor doc as canonical source#143

Merged
vbatts merged 1 commit intoopencontainers:masterfrom
philips:remove-external-doc-dependency
Jun 16, 2016
Merged

manifest: use descriptor doc as canonical source#143
vbatts merged 1 commit intoopencontainers:masterfrom
philips:remove-external-doc-dependency

Conversation

@philips
Copy link
Contributor

@philips philips commented Jun 15, 2016

Right now manifest is relying on the external Docker docs to explain the
digest format. Instead use the OCI Descriptor docs.

Signed-off-by: Brandon Philips brandon.philips@coreos.com

Right now manifest is relying on the external Docker docs to explain the
digest format. Instead use the OCI Descriptor docs.

Signed-off-by: Brandon Philips <brandon.philips@coreos.com>
@runcom
Copy link
Member

runcom commented Jun 15, 2016

+1

@philips philips added this to the v1.0.0-rc1 milestone Jun 15, 2016
@jonboulle
Copy link
Contributor

jonboulle commented Jun 16, 2016

LGTM

Approved with PullApprove

@philips
Copy link
Contributor Author

philips commented Jun 16, 2016

LGTM

Approved with PullApprove

1 similar comment
@vbatts
Copy link
Member

vbatts commented Jun 16, 2016

LGTM

Approved with PullApprove

@vbatts vbatts merged commit 656697a into opencontainers:master Jun 16, 2016
wking added a commit to wking/image-spec that referenced this pull request Jul 2, 2016
1149fac (manifest: use descriptor doc as canonical source,
2016-06-15, opencontainers#143) was a step in this direction.  But we can DRY things
up a bit more by punting those three properties entirely and just
saying "it's a descriptor", and then listing extensions, if any.

Signed-off-by: W. Trevor King <wking@tremily.us>
wking added a commit to wking/image-spec that referenced this pull request Jul 2, 2016
1149fac (manifest: use descriptor doc as canonical source,
2016-06-15, opencontainers#143) was a step in this direction.  But we can DRY things
up a bit more by punting those three properties entirely and just
saying "it's a descriptor", and then listing extensions, if any.

Signed-off-by: W. Trevor King <wking@tremily.us>
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.

4 participants