-
-
Notifications
You must be signed in to change notification settings - Fork 28
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
Create image workflow #345
base: main
Are you sure you want to change the base?
Changes from 14 commits
90351d7
a0c41ef
d1aea92
ff1931f
d4e924c
33c94f7
14f5e25
faa6d2b
be67a65
36e8c66
39e4488
ce1f71c
51ae39f
333aed7
aab3184
616d332
f1c91fb
c4fc907
c4d84d4
ea80265
df4aa71
0ca571a
1aa607a
8252409
ac0417f
c0fab0b
b540d32
2b63502
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,27 @@ | ||
name: Docker Image Build | ||
|
||
on: | ||
workflow_run: | ||
workflows: ["CWL Viewer Build"] | ||
branches: [main] | ||
types: | ||
- completed | ||
|
||
jobs: | ||
build-image: | ||
runs-on: ubuntu-20.04 | ||
|
||
steps: | ||
- uses: actions/[email protected] | ||
with: | ||
ref: ${{ github.event.pull_request.head.ref }} | ||
repository: ${{ github.event.pull_request.head.repo.full_name }} | ||
|
||
- name: Docker Image Build and Push | ||
run: | | ||
DATE=$(date +%s) | ||
GIT_HASH=$(git rev-parse --short "$GITHUB_SHA") | ||
VERSION= ${GIT_HASH}-${DATE} | ||
docker login -u ${{ secrets.DOCKER_HUB_USERNAME }} -p ${{ secrets.DOCKER_HUB_ACCESS_TOKEN }} | ||
docker build . --file Dockerfile --tag ${{ secrets.DOCKER_HUB_USERNAME }}/cwlviewer:${VERSION} | ||
docker push ${{ secrets.DOCKER_HUB_USERNAME }}/cwlviewer:${VERSION} | ||
Comment on lines
+22
to
+27
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can you review https://github.com/common-workflow-language/cwlviewer/pull/360/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R216 and incorporate the process documented there? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @obasekiosa let me know if you have questions There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. specifically, I would like this to match the tagging conventions and the |
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.