Skip to content

Commit

Permalink
Fix publish to pypi action (#1127)
Browse files Browse the repository at this point in the history
In #1124, I accidentally added a line doing a `pip install .` package
install to the publish flow.

This is silly, we shouldn't need to be installing the package before
building the package for pypi, and it appears to be causing failures
on the action:
https://github.com/keras-team/keras-nlp/actions/runs/5513665846/jobs/10052068366

These were not caught because we don't run the pypi action on each PR.
  • Loading branch information
mattdangerw authored Jul 10, 2023
1 parent d7c23e8 commit 03668b8
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion .github/workflows/publish-to-pypi.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ jobs:
- name: Install dependencies
run: |
pip install -r requirements.txt --progress-bar off
pip install --no-deps -e "." --progress-bar off
- name: Build a binary wheel and a source tarball
run: >-
python pip_build.py
Expand Down

0 comments on commit 03668b8

Please sign in to comment.