|
19 | 19 | runs-on: ubuntu-latest |
20 | 20 | outputs: |
21 | 21 | release: ${{ steps.version.outputs.release }} |
22 | | - js: ${{ steps.js.outputs.release }} |
23 | | - python: ${{ steps.python.outputs.release }} |
24 | 22 | charts: ${{ steps.charts.outputs.release }} |
25 | 23 | template: ${{ steps.template.outputs.release }} |
26 | 24 | itinerary: ${{ steps.itinerary.outputs.itinerary }} |
|
74 | 72 | IS_RELEASE=$(./.github/scripts/is_release.sh) |
75 | 73 | echo "release=$IS_RELEASE" >> "$GITHUB_OUTPUT" |
76 | 74 |
|
77 | | - - name: Check JavaScript SDK Release |
78 | | - id: js |
79 | | - if: steps.version.outputs.release == 'true' |
80 | | - run: | |
81 | | - IS_RELEASE=$(./.github/scripts/is_release_for_package.sh "@e2b/code-interpreter") |
82 | | - echo "release=$IS_RELEASE" >> "$GITHUB_OUTPUT" |
83 | | -
|
84 | | - - name: Check Python SDK Release |
85 | | - id: python |
86 | | - if: steps.version.outputs.release == 'true' |
87 | | - run: | |
88 | | - IS_RELEASE=$(./.github/scripts/is_release_for_package.sh "@e2b/code-interpreter-python") |
89 | | - echo "release=$IS_RELEASE" >> "$GITHUB_OUTPUT" |
90 | | -
|
91 | 75 | - name: Check Charts SDK Release |
92 | 76 | id: charts |
93 | 77 | if: steps.version.outputs.release == 'true' |
@@ -274,29 +258,11 @@ jobs: |
274 | 258 | E2B_API_KEY: ${{ secrets.E2B_PROD_API_KEY }} |
275 | 259 | E2B_DOMAIN: ${{ vars.E2B_DOMAIN }} |
276 | 260 |
|
277 | | - python-tests: |
278 | | - name: Python Tests |
279 | | - needs: [preflight, build-template] |
280 | | - if: (!cancelled()) && |
281 | | - !contains(needs.*.result, 'failure') && |
282 | | - needs.preflight.outputs.template == 'true' |
283 | | - uses: ./.github/workflows/python_tests.yml |
284 | | - secrets: inherit |
285 | | - |
286 | | - js-tests: |
287 | | - name: JS Tests |
288 | | - needs: [preflight, build-template] |
289 | | - if: (!cancelled()) && |
290 | | - !contains(needs.*.result, 'failure') && |
291 | | - needs.preflight.outputs.template == 'true' |
292 | | - uses: ./.github/workflows/js_tests.yml |
293 | | - secrets: inherit |
294 | | - |
295 | 261 | release: |
296 | | - # Every upstream job is listed, not just the tests: a job that fails makes its |
297 | | - # dependents *skip*, and a skipped test job is not a failure — so gating on |
| 262 | + # Every upstream job is listed, not just the last one: a job that fails makes |
| 263 | + # its dependents *skip*, and a skipped job is not a failure — so gating on |
298 | 264 | # `needs.*.result` only works for the jobs this one depends on directly. |
299 | | - needs: [preflight, charts-release, build-docker-image, build-template, python-tests, js-tests] |
| 265 | + needs: [preflight, charts-release, build-docker-image, build-template] |
300 | 266 | if: (!cancelled()) && |
301 | 267 | !contains(needs.*.result, 'failure') && |
302 | 268 | needs.preflight.outputs.release == 'true' |
@@ -438,7 +404,7 @@ jobs: |
438 | 404 | report-failure: |
439 | 405 | # `preflight` included so a failure there is reported too, whether or not |
440 | 406 | # `failure()` looks past this job's direct dependencies. |
441 | | - needs: [preflight, charts-release, build-docker-image, build-template, python-tests, js-tests, release] |
| 407 | + needs: [preflight, charts-release, build-docker-image, build-template, release] |
442 | 408 | if: failure() |
443 | 409 | name: Code Interpreter Release Failed - Slack Notification |
444 | 410 | runs-on: ubuntu-latest |
|
0 commit comments