Skip to content
This repository was archived by the owner on Jul 18, 2023. It is now read-only.

Comments

Artifact authors how to#6

Closed
SteveLasker wants to merge 3 commits intoopencontainers:mainfrom
SteveLasker:artifact-authors-how-to
Closed

Artifact authors how to#6
SteveLasker wants to merge 3 commits intoopencontainers:mainfrom
SteveLasker:artifact-authors-how-to

Conversation

@SteveLasker
Copy link
Contributor

@SteveLasker SteveLasker commented Aug 24, 2019

Ready for review and feedback. Particularly on the artifactType.json section.

@SteveLasker SteveLasker force-pushed the artifact-authors-how-to branch 5 times, most recently from cb17efb to ed2dd09 Compare August 26, 2019 01:19
@SteveLasker SteveLasker marked this pull request as ready for review August 26, 2019 01:21
@SteveLasker SteveLasker force-pushed the artifact-authors-how-to branch 6 times, most recently from 93e99db to 458bdfd Compare August 26, 2019 21:57
@SteveLasker SteveLasker force-pushed the artifact-authors-how-to branch from 458bdfd to 87563ed Compare August 26, 2019 22:08
@SteveLasker SteveLasker force-pushed the artifact-authors-how-to branch 3 times, most recently from ca3d143 to 760bceb Compare August 29, 2019 17:55
Signed-off-by: Steven Lasker <stevenlasker@hotmail.com>

The uniqueness of an artifact is defined by its type. An artifact has a type, which has a collection of layers.

The Artifact is defined as unique by its `manifest.config.mediaType`. Layers are defined by their `layer.config.mediaType`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we have some clarity regarding the suggestion for using + vs. . at the end of a mediatype? For example, when to use .json vs. +json?

Signed-off-by: Steven Lasker <stevenlasker@hotmail.com>
Signed-off-by: Steven Lasker <stevenlasker@hotmail.com>
jdolitsky pushed a commit to bloodorangeio/artifacts that referenced this pull request Feb 9, 2021
@SteveLasker
Copy link
Contributor Author

This PR is now outdated, and covered in the existing docs.

@SteveLasker SteveLasker closed this Jul 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants