Merge pull request #14 from NiftyPET/devel

casperdcl · web-flow · commit 847aeddcd2fe · 2021-01-06T23:21:40.000Z
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -22,10 +22,8 @@ jobs:
         key: pre-commit|${{ env.PYSHA }}|${{ hashFiles('.pre-commit-config.yaml') }}
     - name: dependencies
       run: |
-        pip install -U pre-commit twine setuptools wheel setuptools_scm[toml] ninst scikit-build
+        pip install -U pre-commit
         sudo apt-get install -yqq clang-format
-    - run: PATHTOOLS=$HOME/NiftyPET_tools python setup.py sdist
-    - run: twine check dist/*
     - uses: reviewdog/action-setup@v1
     - name: comment
       run: |
@@ -42,19 +40,23 @@ jobs:
     - run: pre-commit run -a --show-diff-on-failure
   test:
     if: github.event_name != 'pull_request' || github.head_ref != 'devel'
-    runs-on: [self-hosted, cuda]
+    runs-on: [self-hosted, cuda, python]
     name: Test
     steps:
     - uses: actions/checkout@v2
       with:
         fetch-depth: 0
+    - name: Run setup-python
+      run: setup-python -p3.7 -m ${GITHUB_RUN_ID}.${GITHUB_RUN_NUMBER}
     - run: pip install -U -e .[dev]
     - run: pytest
     - run: codecov
       env:
         CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
+    - name: Post Run setup-python
+      run: setup-python -p3.7 -m ${GITHUB_RUN_ID}.${GITHUB_RUN_NUMBER} -r
+      if: ${{ always() }}
   deploy:
-    if: github.event_name == 'push' && startsWith(github.event.ref, 'refs/tags')
     needs: [check, test]
     name: PyPI Deploy
     runs-on: ubuntu-latest
@@ -63,22 +65,24 @@ jobs:
       with:
         fetch-depth: 0
     - uses: actions/setup-python@v2
-      with:
-        python-version: '3.x'
     - run: pip install -U twine setuptools wheel setuptools_scm[toml] ninst scikit-build
     - run: PATHTOOLS=$HOME/NiftyPET_tools python setup.py sdist
-    - run: twine upload dist/*
+    - run: twine check dist/*
+    - if: github.event_name == 'push' && startsWith(github.event.ref, 'refs/tags')
+      run: twine upload dist/*
       env:
         TWINE_USERNAME: __token__
         TWINE_PASSWORD: ${{ secrets.PYPI_TOKEN }}
         skip_existing: true
-    - id: collect_assets
+    - if: github.event_name == 'push' && startsWith(github.event.ref, 'refs/tags')
+      id: collect_assets
       name: Collect assets
       run: |
         echo "::set-output name=asset_path::$(ls dist/*.tar.gz)"
         echo "::set-output name=asset_name::$(basename dist/*.tar.gz)"
         git log --pretty='format:%d%n- %s%n%b---' $(git tag --sort=v:refname | tail -n2 | head -n1)..HEAD > _CHANGES.md
-    - id: create_release
+    - if: github.event_name == 'push' && startsWith(github.event.ref, 'refs/tags')
+      id: create_release
       uses: actions/create-release@v1
       env:
         GITHUB_TOKEN: ${{ secrets.GH_TOKEN }}
@@ -87,7 +91,8 @@ jobs:
         release_name: nimpa ${{ github.ref }} beta
         body_path: _CHANGES.md
         draft: true
-    - uses: actions/upload-release-asset@v1
+    - if: github.event_name == 'push' && startsWith(github.event.ref, 'refs/tags')
+      uses: actions/upload-release-asset@v1
       env:
         GITHUB_TOKEN: ${{ secrets.GH_TOKEN }}
       with:
