assapin commented on a change in pull request #12:
URL: https://github.com/apache/incubator-liminal/pull/12#discussion_r567633918
##########
File path: .github/workflows/publish-package.yml
##########
@@ -0,0 +1,31 @@
+name: Publish package
+
+on:
+ push:
+ tags:
+ - '**'
+
+jobs:
+ publish-package:
+ name: Publish package
+ runs-on: ubuntu-latest
+ steps:
+ - uses: actions/checkout@v1
+ - name: Set env
+ run: echo "LIMINAL_VERSION=${GITHUB_REF#refs/*/}" >> $GITHUB_ENV
+ - name: Set up Python
+ uses: actions/setup-python@v1
+ with:
+ python-version: '3.6'
+ - name: Install dependencies for setup
+ run: |
+ python -m pip install --upgrade pip
+ pip install setuptools wheel twine
+ - name: Build and publish
+ env:
+ TWINE_USERNAME: __token__
+ TWINE_PASSWORD: ${{ secrets.PYPI_PASSWORD }}
+ TWINE_REPOSITORY_URL: https://upload.pypi.org/legacy/
+ run: |
+ python setup.py sdist bdist_wheel
Review comment:
test pypi on merge to master seems logical
"real" pypi is only for RC and official versions - triggered manually makes
sense too for now
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
