Skip to content

Commit e90e36a

Browse files
committed
Merge branch 'simplify-workflow' into dev
2 parents f5d2067 + 23e3ce2 commit e90e36a

File tree

1 file changed

+2
-13
lines changed

1 file changed

+2
-13
lines changed

.github/workflows/python-package.yml

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -32,22 +32,11 @@ jobs:
3232
- uses: actions/checkout@v2
3333
- name: Set up Python ${{ matrix.python-version }}
3434
uses: actions/setup-python@v2
35+
# It automatically takes care of pip cache, according to
36+
# https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#about-caching-workflow-dependencies
3537
with:
3638
python-version: ${{ matrix.python-version }}
3739

38-
# Derived from https://github.com/actions/cache/blob/main/examples.md#using-pip-to-get-cache-location
39-
# However, a before-and-after test shows no improvement in this repo,
40-
# possibly because the bottlenect was not in downloading those small python deps.
41-
- name: Get pip cache dir from pip 20.1+
42-
id: pip-cache
43-
run: |
44-
echo "::set-output name=dir::$(pip cache dir)"
45-
- name: pip cache
46-
uses: actions/cache@v2
47-
with:
48-
path: ${{ steps.pip-cache.outputs.dir }}
49-
key: ${{ runner.os }}-py${{ matrix.python-version }}-pip-${{ hashFiles('**/setup.py', '**/requirements.txt') }}
50-
5140
- name: Install dependencies
5241
run: |
5342
python -m pip install --upgrade pip

0 commit comments

Comments
 (0)