Skip to content

Commit 2282e61

Browse files
authored
Merge pull request #997 from consideRatio/main
ci: cache pip only when no container is used
2 parents 7be7eb4 + 94a16d6 commit 2282e61

File tree

2 files changed

+3
-15
lines changed

2 files changed

+3
-15
lines changed

.github/workflows/integration-test.yaml

+3
Original file line numberDiff line numberDiff line change
@@ -126,6 +126,9 @@ jobs:
126126
- uses: actions/setup-python@v5
127127
with:
128128
python-version: "3.10"
129+
cache: pip
130+
cache-dependency-path: |
131+
integration-tests/requirements.txt
129132
130133
# FIXME: The test_bootstrap.py script has duplicated logic to run build
131134
# and start images and run things in them. This makes tests slower,

.github/workflows/unit-test.yaml

-15
Original file line numberDiff line numberDiff line change
@@ -69,21 +69,6 @@ jobs:
6969
python3 -m venv /srv/venv
7070
echo '/srv/venv/bin' >> $GITHUB_PATH
7171
72-
# WARNING: This action loads a cache of pip dependencies based on the
73-
# declared key, and it will save a cache for that key on job
74-
# completion. Make sure to update the key to bust the cache
75-
# properly if you make a change that should influence it.
76-
- name: Load cached Python dependencies
77-
uses: actions/cache@v4
78-
with:
79-
path: /srv/venv/
80-
key: >-
81-
pip-
82-
${{ matrix.runs_on }}-
83-
${{ matrix.ubuntu_version }}-
84-
${{ matrix.python_version }}-
85-
${{ hashFiles('setup.py', 'dev-requirements.txt', '.github/workflows/unit-test.yaml') }}
86-
8772
- name: Install Python dependencies
8873
run: |
8974
pip install -r dev-requirements.txt

0 commit comments

Comments
 (0)