-
Notifications
You must be signed in to change notification settings - Fork 32
83 lines (68 loc) · 2.08 KB
/
lint.yml
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
name: Lint
# Controls when the action will run. Triggers the workflow on push or pull request
# events but only for the main and develop branches.
on:
push:
branches: [ main, develop ]
pull_request:
branches: [ main, develop ]
jobs:
pylint_previous:
name: Generate old pylint score
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
with:
ref: ${{ github.event_name == 'pull_request' && github.event.pull_request.base.sha || github.event.before }}
- name: Set up latest Python
uses: actions/setup-python@v4
with:
python-version: "3.10"
cache: 'pip'
- name: Install dependencies
run: |
python3 -m pip install --upgrade pip
python3 -m pip install -r requirements.txt
- name: Pylint
run: |
make pylint | tail -3 | head -1 > pylint_score.txt
SCORE=$(cat pylint_score.txt)
echo "Score: ${SCORE}"
- name: Save previous pylint score
uses: actions/upload-artifact@v3
with:
name: pylint-score
path: pylint_score.txt
lint:
name: Format and Lint
needs: pylint_previous
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- name: Set up latest Python
uses: actions/setup-python@v4
with:
python-version: "3.10"
cache: 'pip'
- name: Install dependencies
run: |
pip install --upgrade pip
pip install -r requirements.txt
- name: Format
run: |
make format
- name: flake8
run: |
make flake8
- name: Get previous pylint score
uses: actions/download-artifact@v3
with:
name: pylint-score
path: ./pylint_score.txt
- name: Check pylint score didn't decrease
run: |
OLD_SCORE=$(cat pylint_score.txt/pylint_score.txt)
SCORE=$(make pylint | tail -3 | head -1)
echo "Old score: ${OLD_SCORE}"
echo "New score: ${SCORE}"
python3 -c "import sys; sys.exit(1) if float(${OLD_SCORE}) > float(${SCORE}) else None"