Update pypi.yml #51
Update pypi.yml #51
Conversation
| @@ -17,17 +17,32 @@ jobs: | |||
| CIBW_SKIP: cp36-* | |||
| - uses: actions/upload-artifact@v3 | |||
There was a problem hiding this comment.
What about just add poetry build after pypa/cibuildwheel@v2.12.0?
There was a problem hiding this comment.
- name: Poetry build
run: poetry build && mv dist/*.tar.gz wheelhouse/
- uses: actions/upload-artifact@v3
with:
path: wheelhouse/
There was a problem hiding this comment.
We only need single .tar.gz file. If we add this steps after pypa/cibuildwheel@v2.12.0 it will be executed and created per each matrix.os. It is not needed I guess, plus it will take more time.
Well, it will overwrite existing file each time, so not a big problem.
| @@ -17,17 +17,32 @@ jobs: | |||
| CIBW_SKIP: cp36-* | |||
| - uses: actions/upload-artifact@v3 | |||
| - uses: pypa/gh-action-pypi-publish@v1.6.4 | ||
| with: | ||
| user: __token__ | ||
| password: ${{ secrets.pypi_password }} | ||
| packages_dir: wheelhouse/artifact | ||
| packages_dir: dist |
There was a problem hiding this comment.
Does this include all whl files?
There was a problem hiding this comment.
This includes all .whl files generated by pypa/cibuildwheel@v2.12.0
The .whl files are stored to ./wheelhouse dir, then content of ./wheelhouse is uploaded as an artifact with name dist, then in upload job artifacts are downloaded and stored in directories which are named after artifact name e.g. in ./dist for dist artifact and in ./tar for tar artifact. Files created using matrix are merged in artifact, so all should be present as long as they have different names (and they do have different names).
|
Thanks! |
|
And I uploaded tar for 0.2.6 manually, so this can be used in next release |
This solves #50
Added job which creates
.tar.gzdistribution file. The file is stored with wheels just before upload to PyPi