Skip to content

Commit f323137

Browse files
committed
reverted smoketest to use source checkout, fixed test only targets sdist resolution, made wheels no longer depend on smoketest
Signed-off-by: Eric Reinecke <[email protected]>
1 parent e713155 commit f323137

File tree

1 file changed

+3
-8
lines changed

1 file changed

+3
-8
lines changed

.github/workflows/python-package.yml

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,6 @@ jobs:
8989
cmake ${{ github.workspace }}/tests/consumer -DCMAKE_PREFIX_PATH=${{ env.OTIO_INSTALL_DIR }}
9090
9191
py_smoketest_build:
92-
needs: package_sdist
9392
# Ideally this would be ${{ env.GH_COV_OS }} - but github doens't allow it
9493
runs-on: ubuntu-latest
9594
strategy:
@@ -107,11 +106,8 @@ jobs:
107106
steps:
108107
- name: Get tests and requirements from source
109108
uses: actions/checkout@v4
110-
- name: Get sdist source
111-
uses: actions/download-artifact@v5
112109
with:
113-
name: sdist
114-
path: 'sdist'
110+
submodules: "recursive"
115111
- name: Set up Python ${{ matrix.python-version }}
116112
uses: actions/[email protected]
117113
with:
@@ -131,7 +127,7 @@ jobs:
131127
python -m pip install --upgrade -r tests/requirements.txt
132128
- name: Build and Install OpenTimelineIO
133129
run: |
134-
python -m pip install sdist/opentimelineio-*.tar.gz -v --break-system-packages
130+
python -m pip install . -v --break-system-packages
135131
- name: Run tests w/ python coverage
136132
run: make ci-postbuild
137133
# (only on GH_COV_OS and GH_COV_PY)
@@ -201,12 +197,11 @@ jobs:
201197
python -m pip install --break-system-packages --upgrade -r tests/requirements.txt
202198
- name: Build and Install OpenTimelineIO
203199
run: |
204-
python -m pip install sdist/opentimelineio-*.tar.gz -v --break-system-packages
200+
python -m pip install --no-cache-dir --find-links sdist opentimelineio -v --break-system-packages
205201
- name: Run tests w/ python coverage
206202
run: make ci-postbuild
207203

208204
package_wheels:
209-
needs: py_smoketest_build
210205
runs-on: ${{ matrix.os }}
211206
strategy:
212207
matrix:

0 commit comments

Comments
 (0)