-
Notifications
You must be signed in to change notification settings - Fork 1
Add drone config to create artifacts for release #63
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
base: main
Are you sure you want to change the base?
Conversation
2550da3
to
d2991f2
Compare
285396b
to
45b0383
Compare
67fd7fc
to
b383b34
Compare
7f991f6
to
82caf70
Compare
I doubt maybe it's a permission issue (but the same gh token is working for other repos) |
66600ea
to
14be880
Compare
|
Is Docker Image of the app also included in the artifacts? |
docker image should be pushed to the desired container registry (I think we will be using dockerhub) |
yes we are using docker hub, see https://hub.docker.com/u/owncloud. If we follow the same structure/naming conventions like other apps seem to use, it should be https://hub.docker.com/u/owncloud/web-app-dicom-viewer and for the tag dicom-viewer-1.0.0 |
there is some work related to creating docker image in #68, please make sure that in the future creating docker image will be included in drone |
hey @saw-jan any progress on this, respectively if not adding, does this impact rollout? |
14be880
to
b3a2829
Compare
b3a2829
to
e829e1d
Compare
1d1e2d6
to
ca2be87
Compare
ca2be87
to
97d8cde
Compare
|
Description
Create zip file for release.
Related Issue
Motivation and Context
Screenshots (if appropriate):
Types of changes
Checklist: