Skip to content

Commit 8d9937e

Browse files
committed
ci: clean test-job.yml Job of CI/CD Pipeline
1 parent e41cee0 commit 8d9937e

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

.github/workflows/test-job.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
name: "Test Job"
22

3+
# Reusable Workflow for Testing Python Package
4+
# Designed to be the CI componenent in a CI -> CD Pipeline
5+
36
on:
47
workflow_call:
58
inputs:
@@ -60,7 +63,6 @@ jobs:
6063
# COVERAGE_ARTIFACT: ${{ steps.set_coverage_artifact.outputs.COVERAGE_ARTIFACT }}
6164
env:
6265
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
63-
WHEELS_PIP_DIR: "wheels-pip"
6466
# --cov-report=html:{envdir}/htmlcov \
6567
PYTEST_ARGS: >
6668
-ra --cov --cov-report=term-missing \
@@ -155,7 +157,6 @@ jobs:
155157
- name: "Export exact 'Prod + Test' Dependencies in requirements.txt format"
156158
run: uv export --no-emit-project --no-dev --extra test --frozen --format requirements-txt -o prod+test.txt
157159

158-
159160
######## PHASE 1: INSTALL CODE AND RUN SANITY CHECKS ########
160161

161162
# 1. CREATE VENV and INSTALL 'Prod + Test' Dependencies
@@ -298,7 +299,6 @@ jobs:
298299
- name: "Upload Test Coverage as Artifacts"
299300
uses: actions/upload-artifact@v4
300301
with:
301-
# TODO: implement mechanism for uploading test reports on separate artifact names. then codecov host upload job should be able somehow to get all of them. then change below from true to false
302302
overwrite: false
303303
name: coverage-${{ matrix.platform }}-${{ matrix.python-version }}.xml
304304
path: coverage-${{ matrix.platform }}-${{ matrix.python-version }}.xml

0 commit comments

Comments
 (0)