From a189e2618f096ecf496d58ec28f28298f5e5119a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Aleix=20Conchillo=20Flaqu=C3=A9?= <aleix@daily.co>
Date: Tue, 19 Mar 2024 15:31:03 -0700
Subject: [PATCH] github: source venv in every step

---
 .github/workflows/tests.yaml | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/tests.yaml b/.github/workflows/tests.yaml
index 5de8faee6..4222e1e8f 100644
--- a/.github/workflows/tests.yaml
+++ b/.github/workflows/tests.yaml
@@ -29,25 +29,30 @@ jobs:
       - name: Cache virtual environment
         uses: actions/cache@v3
         with:
-          key: venv-${{ runner.os }}-${{ steps.setup_python.outputs.python-version}}
+          # TODO: we are hashing requirements.txt but that doesn't contain all
+          # our dependencies pinned.
+          key: venv-${{ runner.os }}-${{ steps.setup_python.outputs.python-version}}-${{ hashFiles('requirements.txt') }}
           path: .venv
       - name: Install system packages
         run: sudo apt-get install -y portaudio19-dev
       - name: Setup virtual environment
         run: |
           python -m venv .venv
-          source .venv/bin/activate
       - name: Install basic Python dependencies
         run: |
+          source .venv/bin/activate
           python -m pip install --upgrade pip
           pip install -r requirements.txt
       - name: Build project
         run: |
+          source .venv/bin/activate
           python -m build
       - name: Install project and other Python dependencies
         run: |
+          source .venv/bin/activate
           pip install --editable .
       - name: Test with pytest
         run: |
+          source .venv/bin/activate
           pip install pytest
           pytest --doctest-modules --ignore-glob="*to_be_updated*" src tests