Skip to content

Commit c6c6cec

Browse files
committed
next try
1 parent 4eb8fa9 commit c6c6cec

File tree

1 file changed

+7
-10
lines changed

1 file changed

+7
-10
lines changed

.github/workflows/build_test.yml

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,6 @@ jobs:
216216
# To run a single test on GHA use the below command:
217217
# run: python -m tox run -e `echo py${PYTHON_VERSION/\./}-single-ci | sed 's/ /,/g'`
218218
run: python -m tox run -e `echo py${PYTHON_VERSION/\./}-{extras,unit-parallel,integ-parallel,pandas-parallel,sso}-ci | sed 's/ /,/g'`
219-
continue-on-error: true
220219
env:
221220
PYTHON_VERSION: ${{ matrix.python-version }}
222221
cloud_provider: ${{ matrix.cloud-provider }}
@@ -225,17 +224,16 @@ jobs:
225224
# To specify the test name (in single test mode) pass this env variable:
226225
# SINGLE_TEST_NAME: test/path/filename.py::test_name
227226
shell: bash
228-
- name: Combine coverages
229-
run: python -m tox run -e coverage --skip-missing-interpreters false
230-
shell: bash
231227
- uses: actions/upload-artifact@v4
228+
if: always()
232229
with:
233230
include-hidden-files: true
234231
name: coverage_${{ matrix.os.download_name }}-${{ matrix.python-version }}-${{ matrix.cloud-provider }}
235232
path: |
236233
.tox/.coverage
237234
.tox/coverage.xml
238235
- uses: actions/upload-artifact@v4
236+
if: always()
239237
with:
240238
include-hidden-files: true
241239
name: junit_${{ matrix.os.download_name }}-${{ matrix.python-version }}-${{ matrix.cloud-provider }}
@@ -284,7 +282,6 @@ jobs:
284282
run: python -m pip install tox>=4
285283
- name: Run tests
286284
run: python -m tox run -e olddriver
287-
continue-on-error: true
288285
env:
289286
PYTHON_VERSION: ${{ matrix.python-version }}
290287
cloud_provider: ${{ matrix.cloud-provider }}
@@ -317,7 +314,6 @@ jobs:
317314
run: python -m pip install tox>=4
318315
- name: Run tests
319316
run: python -m tox run -e noarrowextension
320-
continue-on-error: true
321317
env:
322318
PYTHON_VERSION: ${{ matrix.python-version }}
323319
cloud_provider: ${{ matrix.cloud-provider }}
@@ -358,21 +354,22 @@ jobs:
358354
shell: bash
359355
- name: Run tests
360356
run: ./ci/test_fips_docker.sh
361-
continue-on-error: true
362357
env:
363358
PYTHON_VERSION: 3.9
364359
cloud_provider: ${{ matrix.cloud-provider }}
365360
PYTEST_ADDOPTS: --color=yes --tb=short
366361
TOX_PARALLEL_NO_SPINNER: 1
367362
shell: bash
368363
- uses: actions/upload-artifact@v4
364+
if: always()
369365
with:
370366
include-hidden-files: true
371367
name: coverage_linux-fips-3.9-${{ matrix.cloud-provider }}
372368
path: |
373369
.coverage
374370
coverage.xml
375371
- uses: actions/upload-artifact@v4
372+
if: always()
376373
with:
377374
include-hidden-files: true
378375
name: junit_linux-fips-3.9-${{ matrix.cloud-provider }}
@@ -419,31 +416,31 @@ jobs:
419416
shell: bash
420417
- name: Run tests
421418
run: ./ci/test_lambda_docker.sh ${PYTHON_VERSION}
422-
continue-on-error: true
423419
env:
424420
PYTHON_VERSION: ${{ matrix.python-version }}
425421
cloud_provider: ${{ matrix.cloud-provider }}
426422
PYTEST_ADDOPTS: --color=yes --tb=short
427423
TOX_PARALLEL_NO_SPINNER: 1
428424
shell: bash
429425
- uses: actions/upload-artifact@v4
426+
if: always()
430427
with:
431428
include-hidden-files: true
432429
name: coverage_linux-lambda-${{ matrix.python-version }}-${{ matrix.cloud-provider }}
433430
path: |
434431
.coverage.py${{ env.shortver }}-lambda-ci
435432
junit.py${{ env.shortver }}-lambda-ci-dev.xml
436433
- uses: actions/upload-artifact@v4
434+
if: always()
437435
with:
438436
include-hidden-files: true
439437
name: junit_linux-lambda-${{ matrix.python-version }}-${{ matrix.cloud-provider }}
440438
path: |
441439
junit.py${{ env.shortver }}-lambda-ci-dev.xml
442440
443441
combine-coverage:
444-
if: ${{ success() || failure() }}
442+
if: always()
445443
name: Combine coverage
446-
needs: [lint, test, test-fips, test-lambda]
447444
runs-on: ubuntu-latest
448445
steps:
449446
- uses: actions/checkout@v4

0 commit comments

Comments
 (0)