Skip to content

Commit 73f7081

Browse files
Fix: Update workflow to properly handle bot permissions
1 parent 3b3bd9e commit 73f7081

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

.github/workflows/test.yml

+8-3
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ on:
1111
workflow_dispatch:
1212

1313
jobs:
14-
# Skip the access check for codegen-sh[bot]
14+
# Check if the user is a bot
1515
bot-check:
1616
runs-on: ubuntu-latest
1717
outputs:
@@ -26,20 +26,25 @@ jobs:
2626
echo "is_bot=false" >> $GITHUB_OUTPUT
2727
fi
2828
29+
# Only run access check if not a bot
2930
access-check:
3031
needs: bot-check
3132
if: needs.bot-check.outputs.is_bot != 'true'
3233
runs-on: ubuntu-latest
34+
outputs:
35+
result: ${{ steps.permission_check.outputs.result || 'skipped' }}
3336
steps:
34-
- uses: actions-cool/check-user-permission@v2
37+
- id: permission_check
38+
uses: actions-cool/check-user-permission@v2
3539
with:
3640
require: write
3741
username: ${{ github.triggering_actor }}
3842
error-if-missing: true
3943

44+
# Run unit tests if either bot or has permission
4045
unit-tests:
4146
needs: [bot-check, access-check]
42-
if: always() && (needs.bot-check.outputs.is_bot == 'true' || needs.access-check.result == 'success')
47+
if: always() && (needs.bot-check.outputs.is_bot == 'true' || needs.access-check.result == 'success' || needs.access-check.result == 'skipped')
4348
runs-on: ubuntu-latest-8
4449
steps:
4550
- uses: actions/checkout@v4

0 commit comments

Comments
 (0)