Cache dependency installs in CI (#833)

bdraco · web-flow · commit 0bf4f7537a04 · 2021-06-24T23:07:12.000-10:00
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -15,19 +15,34 @@ jobs:
       matrix:
         os: [ubuntu-latest, macos-latest, windows-latest]
         python-version: [3.6, 3.7, 3.8, 3.9, pypy3]
+        include:
+          - os: ubuntu-latest
+            venvcmd: . env/bin/activate
+          - os: macos-latest
+            venvcmd: . env/bin/activate
+          - os: windows-latest
+            venvcmd: env\Scripts\Activate.ps1
     steps:
-    - uses: actions/checkout@v2
-    - name: Set up Python ${{ matrix.python-version }}
-      uses: actions/setup-python@v2
-      with:
-        python-version: ${{ matrix.python-version }}
-    - name: Install dependencies
-      run: |
-        pip install --upgrade -r requirements-dev.txt
-        pip install .
-    - name: Install pytest-github-actions-annotate-failures plugin
-      run: pip install pytest-github-actions-annotate-failures        
-    - name: Run tests
-      run: make ci
-    - name: Report coverage to Codecov
-      uses: codecov/codecov-action@v1
+      - uses: actions/checkout@v2
+      - uses: actions/cache@v2
+        id: cache
+        with:
+          path: env
+          key: ${{ runner.os }}-pip-${{ matrix.python-version }}-${{ hashFiles('**/requirements-dev.txt') }}-${{ hashFiles('**/Makefile') }}
+          restore-keys: |
+            ${{ runner.os }}-pip-
+      - name: Set up Python ${{ matrix.python-version }}
+        uses: actions/setup-python@v2
+        with:
+          python-version: ${{ matrix.python-version }}
+      - name: Install dependencies
+        run: |
+          python -m venv env
+          ${{ matrix.venvcmd }}
+          pip install --upgrade -r requirements-dev.txt pytest-github-actions-annotate-failures
+      - name: Run tests
+        run: |
+          ${{ matrix.venvcmd }}
+          make ci
+      - name: Report coverage to Codecov
+        uses: codecov/codecov-action@v1
diff --git a/Makefile b/Makefile
@@ -13,7 +13,7 @@ endif
 virtualenv: ./env/requirements.built
 
 env:
-	virtualenv env
+	python -m venv env
 
 ./env/requirements.built: env requirements-dev.txt
 	./env/bin/pip install -r requirements-dev.txt
@@ -48,3 +48,4 @@ test_coverage:
 
 autopep8:
 	autopep8 --max-line-length=$(MAX_LINE_LENGTH) -i setup.py examples zeroconf
+
diff --git a/requirements-dev.txt b/requirements-dev.txt
@@ -2,7 +2,7 @@ autopep8
 black;implementation_name=="cpython"
 coveralls
 coverage
-# Version restricted because of https://github.com/PyCQA/pycodestyle/issues/741
+# Version restricted because of https://github.com/PyCQA/pycodestyle/issues/741 - is fixed
 flake8>=3.6.0
 flake8-import-order
 ifaddr
