Skip to content

Commit 5eaec6c

Browse files
fix: skip permission check for bot users in GitHub Actions workflow
1 parent 36ebc9b commit 5eaec6c

1 file changed

Lines changed: 8 additions & 3 deletions

File tree

.github/workflows/test.yml

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,13 @@ jobs:
1919
require: write
2020
username: ${{ github.triggering_actor }}
2121
error-if-missing: true
22+
# Skip this check for bot users
23+
if: ${{ !endsWith(github.triggering_actor, '[bot]') }}
2224

2325
unit-tests:
26+
# Run if access-check is successful or skipped (for bot users)
2427
needs: access-check
28+
if: ${{ always() && (needs.access-check.result == 'success' || needs.access-check.result == 'skipped') }}
2529
runs-on: ubuntu-latest-8
2630
steps:
2731
- uses: actions/checkout@v4
@@ -50,15 +54,15 @@ jobs:
5054
codemod-tests:
5155
needs: access-check
5256
# TODO: re-enable when this check is a develop required check
53-
if: false
57+
if: ${{ always() && (needs.access-check.result == 'success' || needs.access-check.result == 'skipped') && false }}
5458
runs-on: ubuntu-latest-32
5559
strategy:
5660
matrix:
5761
sync_graph: [ true, false ]
5862
size: [ small, large ]
5963
exclude:
6064
# Exclude large codemod tests when not needed
61-
- size: ${{(contains(github.event.pull_request.labels.*.name, 'big-codemod-tests') || github.event_name == 'push' || github.event_name == 'workflow_dispatch') && 'kevin' || 'large'}}
65+
- size: ${{(contains(github.event.pull_request.labels.*.name, 'big-codemod-tests') || github.event_name == 'push' || github.event_name == 'workflow_dispatch') && 'kevin' || 'large')}
6266
- size: large
6367
sync_graph: true
6468
concurrency:
@@ -91,7 +95,7 @@ jobs:
9195

9296
parse-tests:
9397
needs: access-check
94-
if: contains(github.event.pull_request.labels.*.name, 'parse-tests') || github.event_name == 'push' || github.event_name == 'workflow_dispatch'
98+
if: ${{ always() && (needs.access-check.result == 'success' || needs.access-check.result == 'skipped') && (contains(github.event.pull_request.labels.*.name, 'parse-tests') || github.event_name == 'push' || github.event_name == 'workflow_dispatch') }}
9599
runs-on: ubuntu-latest-32
96100
steps:
97101
- uses: actions/checkout@v4
@@ -162,6 +166,7 @@ jobs:
162166
163167
integration-tests:
164168
needs: access-check
169+
if: ${{ always() && (needs.access-check.result == 'success' || needs.access-check.result == 'skipped') }}
165170
runs-on: ubuntu-latest-16
166171
steps:
167172
- uses: actions/checkout@v4

0 commit comments

Comments
 (0)