-
Notifications
You must be signed in to change notification settings - Fork 694
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
Provide a decision tree for artifacts #1101
Conversation
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.
(explicit NACK, for the avoidance of doubt)
16e207f
to
72b3fb0
Compare
72b3fb0
to
dab402d
Compare
Signed-off-by: Brandon Mitchell <git@bmitch.net> Helped-by: Dasith Wijesiriwardena <dasiths@hotmail.com>
dab402d
to
e6a75e6
Compare
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 but IANAM
When this is done, the `config.mediaType` value should not be a known OCI image config [media type](media-types.md). | ||
Historically, due to registry limitations, some tools have created non-OCI conformant artifacts using the `application/vnd.oci.image.config.v1+json` value for `config.mediaType` and values specific to the artifact in `layer[*].mediaType`. |
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.
👍 on the updated language here ❤️
This adds a decision tree to the artifact guidance to indicate when the artifactType field should and should not be used. I believe this closes #1100.