Skip to content

Commit

Permalink
Repair the release
Browse files Browse the repository at this point in the history
The repo was not properly quoted and the github.ref has another format with direct push.
  • Loading branch information
Michael Vasseur committed Jan 16, 2024
1 parent 8944e11 commit 6232213
Show file tree
Hide file tree
Showing 6 changed files with 13 additions and 7 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/build-contributor-container-PR.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ on:

jobs:
pr-contributor:
if: ${{ github.repository != 'domjudge/domjudge-packaging' || github.ref != 'main' }}
if: ${{ github.repository != 'domjudge/domjudge-packaging' || github.ref != 'refs/heads/main' }}
runs-on: ubuntu-latest
steps:
- name: Checkout
Expand Down Expand Up @@ -39,4 +39,5 @@ jobs:
platforms: linux/amd64,linux/arm64
push: true
tags: ${{ steps.meta.outputs.tags }}
labels: ${{ steps.meta.outputs.labels }}
labels: ${{ steps.meta.outputs.labels }}

3 changes: 2 additions & 1 deletion .github/workflows/build-contributor-container-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ on:

jobs:
release-contributor:
if: ${{ github.repository == 'domjudge/domjudge-packaging' && github.ref == 'main' }}
if: ${{ github.repository == 'domjudge/domjudge-packaging' && github.ref == 'refs/heads/main' }}
runs-on: ubuntu-latest
steps:
- name: Checkout
Expand All @@ -33,3 +33,4 @@ jobs:
platforms: linux/amd64,linux/arm64
push: true
tags: domjudge/domjudge-contributor:latest

2 changes: 1 addition & 1 deletion .github/workflows/build-domjudge-container-PR.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ env:

jobs:
pr-domjudge:
if: ${{ github.repository != 'domjudge/domjudge-packaging' || github.ref != 'main' }}
if: ${{ github.repository != 'domjudge/domjudge-packaging' || github.ref != 'refs/heads/main' }}
runs-on: ubuntu-latest
steps:
- name: Checkout
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/build-domjudge-container-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ env:

jobs:
release-domjudge:
if: ${{ github.repository == 'domjudge/domjudge-packaging' && github.ref == 'main' }}
if: ${{ github.repository == 'domjudge/domjudge-packaging' && github.ref == 'refs/heads/main' }}
runs-on: ubuntu-latest
steps:
- name: Checkout
Expand Down Expand Up @@ -63,3 +63,4 @@ jobs:
docker push domjudge/$IMG:latest
fi
done
3 changes: 2 additions & 1 deletion .github/workflows/build-gitlab-container-PR.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ on:

jobs:
pr-gitlab:
if: ${{ github.repository != 'domjudge/domjudge-packaging' || github.ref != 'main' }}
if: ${{ github.repository != 'domjudge/domjudge-packaging' || github.ref != 'refs/heads/main' }}
name: PR GitLab image
runs-on: ubuntu-latest
permissions:
Expand Down Expand Up @@ -40,3 +40,4 @@ jobs:
push: true
tags: ${{ steps.meta.outputs.tags }}
labels: ${{ steps.meta.outputs.labels }}

4 changes: 3 additions & 1 deletion .github/workflows/build-gitlab-container-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,10 @@ on:
push:
branches:
- main

jobs:
build-gitlab:
if: ${{ github.repository == domjudge/domjudge-packaging && github.ref == main }}
if: ${{ github.repository == 'domjudge/domjudge-packaging' && github.ref == 'refs/heads/main' }}
name: Build GitLab image
runs-on: ubuntu-latest
permissions:
Expand Down Expand Up @@ -38,3 +39,4 @@ jobs:
push: true
tags: "domjudge/gitlabci:22.04"
labels: ${{ steps.meta.outputs.labels }}

0 comments on commit 6232213

Please sign in to comment.