Skip to content

Commit 5e0e785

Browse files
authored
fix(ci): ci "race-condition" (#1499)
See <https://github.com/teutonet/teutonet-helm-charts/actions/runs/15487318332>, somehow these expressions are still evaluated, even though the `if` should skip it
1 parent 036125b commit 5e0e785

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

.github/workflows/linter.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,15 +15,15 @@ jobs:
1515
permissions:
1616
pull-requests: read
1717
with:
18-
pr_number: ${{ github.event.pull_request.number }}
18+
pr_number: ${{ github.event.pull_request.number || 0 }}
1919
getChangedChartsMQ:
2020
uses: ./.github/workflows/get-changed-charts-merge-queue.yaml
2121
if: ${{ github.event.merge_group != null }}
2222
permissions:
2323
contents: read
2424
with:
25-
base_commit: ${{ github.event.merge_group.base_sha }}
26-
head_commit: ${{ github.event.merge_group.head_sha }}
25+
base_commit: ${{ github.event.merge_group.base_sha || '' }}
26+
head_commit: ${{ github.event.merge_group.head_sha || '' }}
2727
lint-helm-chart:
2828
name: lint helm chart
2929
permissions:

.github/workflows/validate-readme.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,15 +18,15 @@ jobs:
1818
permissions:
1919
pull-requests: read
2020
with:
21-
pr_number: ${{ github.event.pull_request.number }}
21+
pr_number: ${{ github.event.pull_request.number || 0 }}
2222
getChangedChartsMQ:
2323
uses: ./.github/workflows/get-changed-charts-merge-queue.yaml
2424
if: ${{ github.event.merge_group != null }}
2525
permissions:
2626
contents: read
2727
with:
28-
base_commit: ${{ github.event.merge_group.base_sha }}
29-
head_commit: ${{ github.event.merge_group.head_sha }}
28+
base_commit: ${{ github.event.merge_group.base_sha || '' }}
29+
head_commit: ${{ github.event.merge_group.head_sha || '' }}
3030
validateREADMEChanges:
3131
if: ${{ always() && contains(needs.*.result, 'success') && !contains(needs.*.result, 'failure') }}
3232
name: Validate README changes

0 commit comments

Comments
 (0)