Skip to content

Commit 82b4566

Browse files
authored
Merge pull request #7611 from webknjaz/testing/gh-actions-ci-cd-linters
Adjust GitHub Actions CI/CD linters
2 parents cc74940 + f12c417 commit 82b4566

File tree

3 files changed

+61
-16
lines changed

3 files changed

+61
-16
lines changed
Lines changed: 53 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
1-
name: Code quality
1+
name: >-
2+
🤖
3+
Code quality
24
35
on:
46
push:
@@ -10,24 +12,49 @@ on:
1012

1113
jobs:
1214
linters:
13-
name: 🤖
15+
name: >-
16+
${{ matrix.env.TOXENV }}/${{ matrix.python-version }}@${{ matrix.os }}
1417
runs-on: ${{ matrix.os }}
1518
strategy:
1619
# max-parallel: 5
1720
matrix:
21+
python-version:
22+
- 3.8
1823
os:
19-
- ubuntu-18.04
24+
- ubuntu-latest
2025
- windows-latest
2126
- macos-latest
2227
env:
2328
- TOXENV: docs
2429
- TOXENV: lint
30+
- TOXENV: vendoring
31+
32+
env:
33+
TOX_PARALLEL_NO_SPINNER: 1
34+
2535
steps:
2636
- uses: actions/checkout@master
27-
- name: Set up Python ${{ matrix.env.PYTHON_VERSION || 3.8 }}
37+
- name: Set up Python ${{ matrix.python-version }}
2838
uses: actions/setup-python@v1
2939
with:
30-
python-version: ${{ matrix.env.PYTHON_VERSION || 3.8 }}
40+
python-version: ${{ matrix.python-version }}
41+
- name: Log Python version
42+
run: >-
43+
python --version
44+
- name: Log Python location
45+
run: >-
46+
which python
47+
- name: Log Python env
48+
run: >-
49+
python -m sysconfig
50+
- name: Pip cache
51+
uses: actions/cache@v1
52+
with:
53+
path: ~/.cache/pip
54+
key: ${{ runner.os }}-pip-${{ hashFiles('tools/requirements/tests.txt') }}-${{ hashFiles('tools/requirements/docs.txt') }}-${{ hashFiles('tox.ini') }}
55+
restore-keys: |
56+
${{ runner.os }}-pip-
57+
${{ runner.os }}-
3158
- name: set PY
3259
run: echo "::set-env name=PY::$(python -VV | sha256sum | cut -d' ' -f1)"
3360
- uses: actions/cache@v1
@@ -38,16 +65,33 @@ jobs:
3865
run: |
3966
git config --global user.email "pypa-dev@googlegroups.com"
4067
git config --global user.name "pip"
41-
- name: Update setuptools and tox dependencies
42-
run: |
68+
- name: Update setuptools
69+
run: >-
4370
python -m pip install --upgrade setuptools
71+
- name: Install tox
72+
run: >-
4473
python -m pip install --upgrade tox tox-venv
74+
- name: Log the list of packages
75+
run: >-
4576
python -m pip freeze --all
4677
- name: 'Initialize tox envs: ${{ matrix.env.TOXENV }}'
4778
run: >-
48-
python -m tox --notest --skip-missing-interpreters false
79+
python -m
80+
tox
81+
--parallel auto
82+
--notest
83+
--skip-missing-interpreters false
4984
env: ${{ matrix.env }}
85+
- name: Pre-fetch pre-commit hooks
86+
# This is to separate test step from deps install
87+
if: matrix.env.TOXENV == 'lint'
88+
run: >-
89+
.tox/lint/${{ runner.os == 'Windows' && 'Scripts' || 'bin' }}/python -m
90+
pre_commit
91+
install-hooks
5092
- name: Test with tox
5193
run: >-
52-
python -m tox
94+
python -m
95+
tox
96+
--parallel auto
5397
env: ${{ matrix.env }}

docs/html/development/ci.rst

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -79,13 +79,13 @@ Current run tests
7979
Developer tasks
8080
---------------
8181

82-
======== =============== ================ =========== ============
83-
OS docs lint vendoring packages
84-
======== =============== ================ =========== ============
85-
Linux Travis, Github Travis, Github Travis Azure
86-
Windows Azure
87-
MacOS Azure
88-
======== =============== ================ =========== ============
82+
======== =============== ================ ================== ============
83+
OS docs lint vendoring packages
84+
======== =============== ================ ================== ============
85+
Linux Travis, Github Travis, Github Travis, Github Azure
86+
Windows Azure
87+
MacOS Azure
88+
======== =============== ================ ================== ============
8989

9090
Actual testing
9191
--------------
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Test vendoring lint target under GitHub Actions CI/CD.

0 commit comments

Comments
 (0)