Skip to content

Commit 383be3a

Browse files
committed
jsp: If on a PR, just do a Docker build to make sure we can.
- push only on a tag (which will have the right creds) #1007
1 parent f991112 commit 383be3a

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

.github/workflows/build-jsp.yml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,13 +95,23 @@ jobs:
9595
with:
9696
images: ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}
9797
- name: Build and push Docker image
98+
if: github.event_name != 'pull_request'
9899
id: push
99100
uses: docker/build-push-action@f2a1d5e99d037542a71f64918e516c093c6f3fc4
100101
with:
101102
context: UnicodeJsps/
102103
push: true
103104
tags: ${{ steps.meta.outputs.tags }}
104105
labels: ${{ steps.meta.outputs.labels }}
106+
- name: Build and Docker image (PR)
107+
if: github.event_name == 'pull_request'
108+
id: builddocker
109+
uses: docker/build-push-action@f2a1d5e99d037542a71f64918e516c093c6f3fc4
110+
with:
111+
context: UnicodeJsps/
112+
push: false
113+
tags: ${{ steps.meta.outputs.tags }}
114+
labels: ${{ steps.meta.outputs.labels }}
105115
- name: Generate artifact attestation
106116
if: github.event_name != 'pull_request'
107117
uses: actions/attest-build-provenance@v2

0 commit comments

Comments
 (0)