Skip to content

Commit

Permalink
Merge pull request #202 from adfinis-sygroup/fix/release-container-im…
Browse files Browse the repository at this point in the history
…age-do-not-login-on-pr

fix(release-container-image.yaml): do not login on pr
  • Loading branch information
eyenx authored Mar 17, 2021
2 parents a9de5f3 + 8b366d1 commit bca0030
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions .github/workflows/release-container-image.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -50,12 +50,14 @@ jobs:
echo ::set-output name=version::${VERSION}
echo ::set-output name=tags::${TAGS}
echo ::set-output name=created::$(date -u +'%Y-%m-%dT%H:%M:%SZ')
- name: Login to GHCR
uses: docker/login-action@v1
with:
registry: ghcr.io
username: ${{ github.repository_owner }}
password: ${{ secrets.PAT }}
if: ${{ github.event_name != 'pull_request' }}

- name: Build and push
id: docker_build
Expand Down Expand Up @@ -115,12 +117,14 @@ jobs:
echo ::set-output name=version::${VERSION}
echo ::set-output name=tags::${TAGS}
echo ::set-output name=created::$(date -u +'%Y-%m-%dT%H:%M:%SZ')
- name: Login to GHCR
uses: docker/login-action@v1
with:
registry: ghcr.io
username: ${{ github.repository_owner }}
password: ${{ secrets.PAT }}
if: ${{ github.event_name != 'pull_request' }}

- name: Build and push
id: docker_build
Expand Down

0 comments on commit bca0030

Please sign in to comment.