Repository: heavyai/rbc (language: python)

Issue:
Update release scripts



Discussion / hints:

Fix patch:
diff --git a/.github/workflows/eval-notebooks.yml b/.github/workflows/eval-notebooks.yml
deleted file mode 100644
index 61b89c4d..00000000
--- a/.github/workflows/eval-notebooks.yml
+++ /dev/null
@@ -1,60 +0,0 @@
-name: Eval notebooks 
-
-on:
-  # Trigger the workflow on push or pull request,
-  # but only for the main branch
-  push:
-    branches:
-      - master
-  pull_request:
-    branches:
-      - master
-
-jobs:
-  eval-notebooks:
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v2
-
-      - name: Set up Python
-        uses: actions/setup-python@v2
-        with:
-          python-version: 3.8
-
-      - uses: conda-incubator/setup-miniconda@v2
-        with:
-          python-version: 3.8
-          mamba-version: "*"
-          channels: conda-forge
-          environment-file: .conda/environment.yml
-
-      - name: Install additional packages
-        shell: bash -l {0}
-        run: |
-          mamba install -c conda-forge omniscidb nbval ibis-omniscidb matplotlib pandas ibis-framework=2.0
-
-      - name: Start omniscidb server
-        shell: bash -l {0}
-        run: |
-          mkdir data && omnisci_initdb data -f
-          omnisci_server --version
-          omnisci_server --enable-runtime-udf --enable-table-functions 2>&1 > omniscidb-output.txt &
-          sleep 10
-
-      - name: conda config
-        shell: bash -l {0}
-        run: conda config --show
-
-      - name: conda list
-        shell: bash -l {0}
-        run: |
-          conda list
-
-      - name: Execute pytest
-        shell: bash -l {0}
-        run: |
-          pip install -e .
-          pytest -v -rs --nbval notebooks/ -x
-          pkill -f omnisci_server
-          cat omniscidb-output.txt
diff --git a/.github/workflows/pypi.yml b/.github/workflows/pypi.yml
index 5dae6835..350a46a4 100644
--- a/.github/workflows/pypi.yml
+++ b/.github/workflows/pypi.yml
@@ -6,6 +6,7 @@ on:
     tags:
       - 'v*' # Push events to matching v*, i.e. v1.0, v20.15.10
 
+
 jobs:
   build_wheels:
     name: Build wheels on ${{ matrix.os }}
@@ -17,8 +18,16 @@ jobs:
     steps:
       - uses: actions/checkout@v2
 
-      - name: Build wheels
+      - name: Set up Python
+        uses: actions/setup-python@v2
+        with:
+          python-version: 3.9
+
+      # https://cibuildwheel.readthedocs.io/en/stable/options/#build-skip
+      - name: Install buildwheels
         uses: pypa/cibuildwheel@v2.3.1
+        env:  # minimum version is cpython 3.7
+          CIBW_BUILD: "cp37-*64 cp38-*64 cp39-* cp310-*"
 
       - uses: actions/upload-artifact@v2
         with:
@@ -38,7 +47,7 @@ jobs:
           path: dist/*.tar.gz
 
   upload_pypi:
-    needs: [build_wheels, build_sdist]
+    needs: [build_sdist]
     runs-on: ubuntu-latest
     # upload to PyPI on every tag starting with 'v'
     if: github.event_name == 'push' && startsWith(github.event.ref, 'refs/tags/v')
@@ -55,5 +64,5 @@ jobs:
           verify_metadata: true
           verbose: true
           user: __token__
-          password: ${{ secrets.pypi_password }}
-          # To test: repository_url: https://test.pypi.org/legacy/
+          password: ${{ secrets.PYPI_API_TOKEN }}
+          # repository_url: https://test.pypi.org/legacy/


Test patch:
diff --git a/.github/workflows/rbc_test.yml b/.github/workflows/rbc_test.yml
index e5effaa4..8988b495 100644
--- a/.github/workflows/rbc_test.yml
+++ b/.github/workflows/rbc_test.yml
@@ -3,13 +3,15 @@ name: RBC
 on:
   # Trigger the workflow on push or pull request,
   # but only for the main branch
-  push:
-    branches:
-      - master
   pull_request:
     branches:
       - master
 
+# kill any previous running job on a new commit
+concurrency:
+  group: build-and-test-rbc-${{ github.head_ref }}
+  cancel-in-progress: true
+
 jobs:
   lint:
     runs-on: ubuntu-latest
@@ -269,3 +271,52 @@ jobs:
         run: |
           mamba run -n docker docker-compose logs --no-color --tail=10000 -f -t \> omniscidb-docker.log
           cat omniscidb-docker.log
+
+  eval-notebooks:
+      runs-on: ubuntu-latest
+      needs: lint
+
+      steps:
+        - uses: actions/checkout@v2
+
+        - name: Set up Python
+          uses: actions/setup-python@v2
+          with:
+            python-version: 3.8
+
+        - uses: conda-incubator/setup-miniconda@v2
+          with:
+            python-version: 3.8
+            mamba-version: "*"
+            channels: conda-forge
+            environment-file: .conda/environment.yml
+
+        - name: Install additional packages
+          shell: bash -l {0}
+          run: |
+            mamba install -c conda-forge omniscidb nbval ibis-omniscidb matplotlib pandas ibis-framework=2.0
+
+        - name: Start omniscidb server
+          shell: bash -l {0}
+          run: |
+            mkdir data && omnisci_initdb data -f
+            omnisci_server --version
+            omnisci_server --enable-runtime-udf --enable-table-functions 2>&1 > omniscidb-output.txt &
+            sleep 10
+
+        - name: conda config
+          shell: bash -l {0}
+          run: conda config --show
+
+        - name: conda list
+          shell: bash -l {0}
+          run: |
+            conda list
+
+        - name: Execute pytest
+          shell: bash -l {0}
+          run: |
+            pip install -e .
+            pytest -v -rs --nbval notebooks/ -x
+            pkill -f omnisci_server
+            cat omniscidb-output.txt