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

Comments

Adds Implementers of OCI Artifacts#4

Merged
mikebrow merged 4 commits intoopencontainers:masterfrom
SteveLasker:implementors
Jul 13, 2021
Merged

Adds Implementers of OCI Artifacts#4
mikebrow merged 4 commits intoopencontainers:masterfrom
SteveLasker:implementors

Conversation

@SteveLasker
Copy link
Contributor

Signed-off-by: Steven Lasker stevenlasker@hotmail.com

Signed-off-by: Steven Lasker <stevenlasker@hotmail.com>
@SteveLasker
Copy link
Contributor Author

Consumes #3, adds implementers

Signed-off-by: Steven Lasker <stevenlasker@hotmail.com>
SteveLasker referenced this pull request in SteveLasker/artifacts Aug 28, 2019
Signed-off-by: Steven Lasker <stevenlasker@hotmail.com>
SteveLasker referenced this pull request in SteveLasker/artifacts Aug 28, 2019
Signed-off-by: Steven Lasker <stevenlasker@hotmail.com>
SteveLasker referenced this pull request in SteveLasker/artifacts Aug 28, 2019
Signed-off-by: Steven Lasker <stevenlasker@hotmail.com>
Copy link
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

see questions comments

@mikebrow
Copy link
Member

needs rebase

Signed-off-by: Steven Lasker <stevenlasker@hotmail.com>
implementors.md Outdated
@@ -0,0 +1,10 @@
# OCI Artifacts Implementations

Projects or companies currently supporting OCI Artifacts
Copy link
Member

Choose a reason for hiding this comment

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

maybe add... "as previously specified under the OCI Image and Runtime specifications (links)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

While I need to update docker/distribution to be distribution/distribution, are we ok to merge this in? This will be the base for others to add their support.

Copy link
Member

Choose a reason for hiding this comment

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

sure let's just make it distribution/distribution then merge..

jdolitsky added a commit to bloodorangeio/artifacts that referenced this pull request Feb 9, 2021
Copy link
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

.. see comment about making it distribution/distribution

Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
Copy link
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

LGTM

@mikebrow mikebrow merged commit 4db5899 into opencontainers:master Jul 13, 2021
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.

2 participants