Skip to content
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

Replace docker registry with OCI registry #310

Merged
merged 1 commit into from
May 25, 2022

Conversation

AidanDelaney
Copy link
Member

Update language to prefer the term "OCI registry" over "docker registry"

Signed-off-by: Aidan Delaney adelaney21@bloomberg.net

Update language to prefer the term "OCI registry" over "docker registry"

Signed-off-by: Aidan Delaney <adelaney21@bloomberg.net>
@jkutner jkutner requested a review from jromero May 20, 2022 13:35
@jkutner
Copy link
Member

jkutner commented May 20, 2022

@jromero i think under the new governance model, this is waiting solely on your approval. I gave it a +1 since it's mildly distribution related.

@jromero
Copy link
Member

jromero commented May 20, 2022

@jromero i think under the new governance model, this is waiting solely on your approval. I gave it a +1 since it's mildly distribution related.

@jkutner ok. Thank you for the call out. Let me look into this and verify I understand the new process. 😅

@samj1912
Copy link
Member

I believe the correct team here would be the implementation team since they own the platform spec and @natalieparellano might need to approve this.

@samj1912
Copy link
Member

This also shouldn't be targeting main and instead a versioned branch.

@samj1912 samj1912 changed the base branch from main to platform/0.10 May 25, 2022 17:39
@samj1912 samj1912 added this to the Platform 0.10 milestone May 25, 2022
@samj1912 samj1912 requested review from natalieparellano and removed request for jromero May 25, 2022 17:40
@samj1912 samj1912 merged commit beaf449 into buildpacks:platform/0.10 May 25, 2022
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.

None yet

5 participants