fix #939: test all supported Python versions instead of just Python 3.8 and trigger tests on PR

we use conda-incubator/setup-miniconda instead of the default setup-python action

Author: gdementen

.github/workflows/python-package-conda.yml

@@ -1,34 +1,40 @@
-name: Python Package using Conda
+name: Test LArray
 
-on: [push]
+# Triggers the workflow on push or pull request events
+on: [push, pull_request]
 
 jobs:
   build-linux:
+    # name: Test (${{ matrix.python-version }}, ${{ matrix.os }})
+    name: Test on ${{ matrix.python-version }}
+    # runs-on: ${{ matrix.os }}
     runs-on: ubuntu-latest
     strategy:
       max-parallel: 5
+      fail-fast: false
+      matrix:
+        # os: ["ubuntu-latest", "macos-latest", "windows-latest"]
+        python-version: [3.6, 3.7, 3.8, 3.9]
+
+    defaults:
+      run:
+        shell: bash -l {0}
 
     steps:
     - uses: actions/checkout@v2
-    - name: Set up Python 3.8
-      uses: actions/setup-python@v2
+    - name: Set up Python ${{ matrix.python-version }}
+      uses: conda-incubator/setup-miniconda@v2
       with:
-        python-version: 3.8
-    - name: Add conda to system path
-      run: |
-        # $CONDA is an environment variable pointing to the root of the miniconda directory
-        echo $CONDA/bin >> $GITHUB_PATH
-    - name: Install dependencies
-      run: |
-        conda env update --file environment.yml --name base
-    - name: Lint with flake8
-      run: |
-        conda install flake8
-        # stop the build if there are Python syntax errors or undefined names
-        flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
-        # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
-        flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
-    - name: Test with pytest
-      run: |
-        conda install pytest
-        pytest
+        auto-update-conda: true
+        environment-file: environment.yml
+        python-version: ${{ matrix.python-version }}
+    - run: conda info
+    - run: conda list
+    - run: conda config --show
+#    - name: Lint with flake8
+#      run: |
+#        # stop the build if there are Python syntax errors or undefined names
+#        flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
+#        # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
+#        flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
+    - run: pytest

environment.yml

@@ -1,9 +1,7 @@
-name: test-py38
+name: test
 channels:
-  - larray-project
   - defaults
 dependencies:
-  - python=3.8
   - pip
   - numpy>=1.13
   - pandas>=0.20
@@ -12,5 +10,6 @@ dependencies:
   - openpyxl
   - xlsxwriter
   - pytest>=3.5
+  - flake8
   - pip:
     - pytest-flake8