Skip to content

Commit

Permalink
[fix(ci)] Remove poetry from publish pipeline (#110)
Browse files Browse the repository at this point in the history
* Remove mypy step, since it's covered by tox

* Update publish.yml to not use poetry
  • Loading branch information
mattkram authored Jan 9, 2024
1 parent aa5b019 commit 81a92b9
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 15 deletions.
20 changes: 9 additions & 11 deletions .github/workflows/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,16 +16,14 @@ jobs:
uses: actions/setup-python@v5
with:
python-version: "3.10"
- name: Install Poetry
uses: snok/install-poetry@v1
with:
virtualenvs-create: true
virtualenvs-in-project: true
- name: Install package dependencies
run: poetry install --no-dev
- name: Build package
run: poetry build
- name: Publish package
run: |
poetry config pypi-token.pypi ${{ secrets.PYPI_TOKEN }}
poetry publish
python -m pip install --upgrade pip
python -m pip install build twine
- name: Build the package
run: python -m build
- name: Upload to PyPI with twine
run: python -m twine upload ./dist/*
env:
TWINE_USERNAME: __token__
TWINE_PASSWORD: ${{ secrets.PYPI_TOKEN }}
4 changes: 0 additions & 4 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,6 @@ jobs:
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python-version }}
- name: Static type checking
continue-on-error: true
run: |
poetry run mypy .
- name: Install dependencies
run: |
python -m pip install --upgrade pip
Expand Down

0 comments on commit 81a92b9

Please sign in to comment.