|
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