Merge pull request #15 from matsui528/ci

Fix github actions
matsui528 · Jan 3, 2022 · b45472e · b45472e
2 parents 5686b82 + eaa9800
commit b45472e
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -9,7 +9,7 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest, macos-latest, windows-latest]
-        python-version: [3.7, 3.8]
+        python-version: [3.8]  #[3.8, 3.9] # faiss doesn't support py3.9 as of Jan 2022. Should be updated some day.
 
     steps:
     - name: Checkout
@@ -22,17 +22,14 @@ jobs:
         miniconda-version: "latest"
         python-version: ${{ matrix.python-version }}
 
-    - name: Set up Python ${{ matrix.python-version }}
-      uses: actions/setup-python@v2
-      with:
-        python-version: ${{ matrix.python-version }}
-
     - name: Install dependencies
+      shell: bash -l {0}    # to activate conda
       run: |
         pip install pytest
         python setup.py install
         conda install -c pytorch faiss-cpu
         
     - name: Test with pytest
+      shell: bash -l {0}   # to activate conda
       run: |
         make test