Skip to content

Commit

Permalink
Resolved comments raised by @dalonsoa
Browse files Browse the repository at this point in the history
  • Loading branch information
Abelarm committed Oct 20, 2022
1 parent 1f4f56d commit 4b7692e
Showing 1 changed file with 16 additions and 20 deletions.
36 changes: 16 additions & 20 deletions .github/workflows/build_deploy_wheels.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,7 @@ name: Upload Python Package


on:
# release:
# types: [published]
push:
# branches:
# - 'fix_deploy/*'

jobs:

Expand Down Expand Up @@ -38,21 +34,21 @@ jobs:
system-packages: 'gcc-gfortran'
pre-build-command: 'export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/usr/local/lib/; export SOLCORE_WITH_PDD=1'

# - name: Publish wheels to PyPI
# if: startsWith(github.ref, 'refs/tags')
# env:
# TWINE_USERNAME: '__token__'
# TWINE_PASSWORD: ${{ secrets.PYPI_TOKEN }}
# run: |
# twine upload dist/*-manylinux*.whl --skip-existing
- name: Publish wheels to PyPI
if: startsWith(github.ref, 'refs/tags')
env:
TWINE_USERNAME: '__token__'
TWINE_PASSWORD: ${{ secrets.PYPI_TOKEN }}
run: |
twine upload dist/*-manylinux*.whl --skip-existing
- uses: actions/upload-artifact@v3
with:
name: linux_wheels
path: dist/

deploy_wheels_macos_windows:
runs-on: ${{ matrix.os }}
runs-on: c{{ matrix.os }}
strategy:
matrix:
os: [macos-latest, windows-latest]
Expand Down Expand Up @@ -98,15 +94,15 @@ jobs:
export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/usr/local/lib/
python setup.py sdist bdist_wheel
# - name: Publish wheels to PyPI
# if: startsWith(github.ref, 'refs/tags')
# env:
# TWINE_USERNAME: '__token__'
# TWINE_PASSWORD: ${{ secrets.PYPI_TOKEN }}
# run: |
# twine upload dist/*.whl --skip-existing
- name: Publish wheels to PyPI
if: startsWith(github.ref, 'refs/tags')
env:
TWINE_USERNAME: '__token__'
TWINE_PASSWORD: ${{ secrets.PYPI_TOKEN }}
run: |
twine upload dist/*.whl --skip-existing
- uses: actions/upload-artifact@v3
with:
name: OS wheels
name: ${{ matrix.os }}_wheels
path: dist/

0 comments on commit 4b7692e

Please sign in to comment.