-
Notifications
You must be signed in to change notification settings - Fork 151
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
Add a GitHub action based release job #723
Add a GitHub action based release job #723
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.
I have not tested but LGTM
Part of eclipse-che/che#18072 Signed-off-by: Eric Williams <ericwill@redhat.com>
Signed-off-by: Eric Williams <ericwill@redhat.com>
Signed-off-by: Eric Williams <ericwill@redhat.com>
@benoitf do you have the secrets to log into Dockerhub? |
yes I think I can get some |
I can register them tomorrow if it works for you |
Any time before next Wednesday's release works fine, thanks! Also please for the devfile registry, as there is the same PR there. |
I forgot to say but I added the docker.io credentials this morning (my morning :-) |
Signed-off-by: Eric Williams ericwill@redhat.com
What does this PR do?
Calls
make-release.sh
with the necessary parameters to release the che-plugin-registry without using centos-ci. Added build/publish steps to themake-release.sh
script.Screenshot/screencast of this PR
N/A
What issues does this PR fix or reference?
Part of eclipse-che/che#18072
How to test this PR?
N/A
PR Checklist
As the author of this Pull Request I made sure that:
What issues does this PR fix or reference
andHow to test this PR
completedReviewers
Reviewers, please comment how you tested the PR when approving it.