Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

allow slashes in ORGANIZATION field by using... #471

Merged

Conversation

nickboldt
Copy link
Contributor

@nickboldt nickboldt commented Oct 6, 2019

allow slashes in ORGANIZATION field by using @ for sed separators instead of /

Related to eclipse-che/che#13765

Change-Id: I69d8d6c6c3c21531e29deaf68e95a8bd7dad14b7
Signed-off-by: nickboldt nboldt@redhat.com

@che-bot
Copy link
Contributor

che-bot commented Oct 6, 2019

Can one of the admins verify this patch?

@nickboldt
Copy link
Contributor Author

Related to #470 -- need this change to allow the ORGANIZATION field to include both the registry and the organization. Eventually Theia will be publishing builds to quay.io, not docker.io, right? So we'll need this. :)

…tead of /

Change-Id: I69d8d6c6c3c21531e29deaf68e95a8bd7dad14b7
Signed-off-by: nickboldt <nboldt@redhat.com>
@nickboldt nickboldt force-pushed the allow-slashes-in-ORGANIZATION branch from cd0e311 to 380c128 Compare October 6, 2019 01:16
@benoitf
Copy link
Contributor

benoitf commented Oct 6, 2019

ci-build

@nickboldt nickboldt merged commit 38e67b2 into eclipse-che:master Oct 7, 2019
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