|
13 | 13 | jobs:
|
14 | 14 | access-check:
|
15 | 15 | runs-on: ubuntu-latest
|
| 16 | + # Skip the access check entirely if the triggering actor is the Codegen bot |
| 17 | + if: github.triggering_actor != 'codegen-sh[bot]' |
16 | 18 | steps:
|
17 | 19 | - uses: actions-cool/check-user-permission@v2
|
18 | 20 | with:
|
19 | 21 | require: write
|
20 | 22 | username: ${{ github.triggering_actor }}
|
21 | 23 | error-if-missing: true
|
22 |
| - # Allow the codegen bot to run tests |
23 |
| - skip-check-users: codegen-sh[bot] |
24 | 24 |
|
25 | 25 | unit-tests:
|
26 |
| - needs: access-check |
| 26 | + # Update the needs condition to handle the case where access-check is skipped |
| 27 | + needs: [access-check] |
| 28 | + if: always() && (needs.access-check.result == 'success' || needs.access-check.result == 'skipped') |
27 | 29 | runs-on: ubuntu-latest-8
|
28 | 30 | steps:
|
29 | 31 | - uses: actions/checkout@v4
|
@@ -51,16 +53,16 @@ jobs:
|
51 | 53 |
|
52 | 54 | codemod-tests:
|
53 | 55 | needs: access-check
|
54 |
| - # TODO: re-enable when this check is a develop required check |
55 |
| - if: false |
| 56 | + # Update the condition to handle the case where access-check is skipped |
| 57 | + if: (always() && (needs.access-check.result == 'success' || needs.access-check.result == 'skipped')) && false |
56 | 58 | runs-on: ubuntu-latest-32
|
57 | 59 | strategy:
|
58 | 60 | matrix:
|
59 | 61 | sync_graph: [ true, false ]
|
60 | 62 | size: [ small, large ]
|
61 | 63 | exclude:
|
62 | 64 | # Exclude large codemod tests when not needed
|
63 |
| - - 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')} |
64 | 66 | - size: large
|
65 | 67 | sync_graph: true
|
66 | 68 | concurrency:
|
|
93 | 95 |
|
94 | 96 | parse-tests:
|
95 | 97 | needs: access-check
|
96 |
| - 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') |
97 | 99 | runs-on: ubuntu-latest-32
|
98 | 100 | steps:
|
99 | 101 | - uses: actions/checkout@v4
|
@@ -164,6 +166,7 @@ jobs:
|
164 | 166 |
|
165 | 167 | integration-tests:
|
166 | 168 | needs: access-check
|
| 169 | + if: always() && (needs.access-check.result == 'success' || needs.access-check.result == 'skipped') |
167 | 170 | runs-on: ubuntu-latest-16
|
168 | 171 | steps:
|
169 | 172 | - uses: actions/checkout@v4
|
|
0 commit comments