Skip to content

Commit

Permalink
Update publish_pypi.yml
Browse files Browse the repository at this point in the history
- remove '--no-dependencies' from 'pip install peekingduck'
- remove separate 'pip install' above it, to avoid future maintenance overheads
  • Loading branch information
ongtw authored Aug 18, 2022
1 parent eb6e2fe commit ee1def3
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions .github/workflows/publish_pypi.yml
Original file line number Diff line number Diff line change
Expand Up @@ -54,8 +54,7 @@ jobs:
- name: Test installation from Test PyPI
run: |
sleep 300
pip install click pyyaml colorama typeguard
pip install -i https://test.pypi.org/simple/ peekingduck==$VERSION --no-dependencies # no tensorflow
pip install -i https://test.pypi.org/simple/ peekingduck==$VERSION
peekingduck
- name: Publish distribution 📦 to PyPI
# only pushes to actual PyPI if its a tag (i.e. on release: published)
Expand All @@ -76,6 +75,5 @@ jobs:
- name: Test installation from PyPI
run: |
sleep 300
pip install click pyyaml colorama typeguard
pip install peekingduck==$VERSION --no-dependencies # no tensorflow
pip install peekingduck==$VERSION
peekingduck

0 comments on commit ee1def3

Please sign in to comment.