Merge pull request cta-observatory#45 from jlenain/update-ci

Update CI
frnbrun · Feb 12, 2024 · 5680c92 · 5680c92
2 parents fe3bd78 + f8daba8
commit 5680c92
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -1,5 +1,6 @@
 name: CI
 
+# by not building all branches on push, we avoid the duplicated builds in PRs
 on:
   push:
     branches:
@@ -18,11 +19,12 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: [3.9]
-        ctapipe-version: [v0.19.2]
+        python-version: ["3.9", "3.10", "3.11"]
+        ctapipe-version: [v0.19.3]
 
     defaults:
       run:
+        # We need login shells (-l) for micromamba to work.
         shell: bash -leo pipefail {0}
 
     steps:
@@ -62,4 +64,4 @@ jobs:
         run: |
           pytest --cov=ctapipe_io_nectarcam --cov-report=xml
 
-      - uses: codecov/codecov-action@v1
+      - uses: codecov/codecov-action@v3
diff --git a/.github/workflows/deploy.yml b/.github/workflows/deploy.yml
@@ -17,13 +17,14 @@ jobs:
       - name: Set up Python
         uses: actions/setup-python@v4
         with:
-          python-version: 3.8
+          python-version: 3.9
 
       - name: Install dependencies
         run: |
           python --version
           pip install -U pip setuptools wheel setuptools_scm[toml]
           python setup.py sdist bdist_wheel
+
       - name: Publish package
         uses: pypa/gh-action-pypi-publish@release/v1
         with: