Skip to content

Commit 71ae0ec

Browse files
authored
Merge pull request #341 from plotly/run-tests-on-dist
Fix typo in venv activation
2 parents f81e04e + 74c1f36 commit 71ae0ec

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

.github/workflows/release.yml

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,12 +28,10 @@ jobs:
2828
uv pip install ".[dev]"
2929
npm i
3030
shell: bash
31-
- name: Run tests
32-
# Run all tests before generating the distribution
31+
- name: Build package
3332
run: |
3433
source .venv/bin/activate
3534
npm run build
36-
npm run test
3735
shell: bash
3836
- name: Generate distribution
3937
# Run setup.py to generate the distribution
@@ -45,7 +43,7 @@ jobs:
4543
# Run tests again using the generated wheel file
4644
run: |
4745
uv venv test-dist
48-
source dist/bin/activate
46+
source test-dist/bin/activate
4947
WHL_FILE=$(ls dist/*.whl)
5048
uv pip install "${WHL_FILE}[dev]"
5149
npm run test

0 commit comments

Comments
 (0)