From 99ad4e97c5cabe3eea7db33f02b4cdfef655a388 Mon Sep 17 00:00:00 2001 From: Jon Turney Date: Sun, 23 May 2021 22:07:09 +0100 Subject: [PATCH] CI: Improve how images workflow works in forks Use the respository owner name rather than hardcoding 'mesonbuild' Don't run this workflow if docker credentials aren't in secrets (has to be written rather indirectly due to https://github.com/actions/runner/issues/520) --- .github/workflows/images.yml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.github/workflows/images.yml b/.github/workflows/images.yml index 87fae3bc2e0e..0fa54df768e7 100644 --- a/.github/workflows/images.yml +++ b/.github/workflows/images.yml @@ -21,6 +21,9 @@ on: jobs: build: + env: + HAS_DOCKER_CREDENTIALS: ${{ secrets.DOCKER_PASSWORD != '' }} + if: ${{ env.HAS_DOCKER_CREDENTIALS }} name: ${{ matrix.cfg.name }} runs-on: ubuntu-latest strategy: @@ -53,4 +56,4 @@ jobs: # Publish - name: Push the ${{ matrix.cfg.id }} image if: github.event_name == 'push' || github.event_name == 'schedule' - run: docker push mesonbuild/${{ matrix.cfg.id }} + run: docker push ${{ github.repository_owner }}/{{ matrix.cfg.id }}