make linting workflow reflect what is done locally (Makefile).
This commit is contained in:
parent
dc3d96c6a8
commit
753e9c5b15
42
.github/workflows/linting.yml
vendored
42
.github/workflows/linting.yml
vendored
|
@ -1,25 +1,33 @@
|
|||
name: Run Linters
|
||||
name: Linters
|
||||
|
||||
on: [push, pull_request]
|
||||
on:
|
||||
push:
|
||||
branches: [ master ]
|
||||
pull_request:
|
||||
branches: [ master ]
|
||||
|
||||
jobs:
|
||||
mypy:
|
||||
black:
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- name: Checkout
|
||||
uses: actions/checkout@v1
|
||||
- name: Run MyPy python type checker
|
||||
uses: jpetrucciani/mypy-check@master
|
||||
with:
|
||||
path: 'lnbits'
|
||||
|
||||
- uses: actions/checkout@v2
|
||||
- run: sudo apt-get install python3-venv
|
||||
- run: python3 -m venv venv
|
||||
- run: ./venv/bin/pip install black
|
||||
- run: make checkblack
|
||||
prettier:
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- name: Checkout
|
||||
uses: actions/checkout@v1
|
||||
- name: Check JS code formatting convention
|
||||
uses: creyD/prettier_action@v2.2
|
||||
with:
|
||||
dry: True
|
||||
prettier_options: --write lnbits/static/js/** lnbits/core/static/js/** lnbits/extensions/*/templates/**
|
||||
- uses: actions/checkout@v2
|
||||
- run: npm install
|
||||
- run: make checkprettier
|
||||
mypy:
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- uses: actions/checkout@v2
|
||||
- run: sudo apt-get install python3-venv
|
||||
- run: sudo apt-get install libev-dev
|
||||
- run: python3 -m venv venv
|
||||
- run: ./venv/bin/pip install -r requirements.txt
|
||||
- run: ./venv/bin/pip install mypy
|
||||
- run: make mypy
|
||||
|
|
2
.github/workflows/tests.yml
vendored
2
.github/workflows/tests.yml
vendored
|
@ -8,7 +8,7 @@ jobs:
|
|||
runs-on: ubuntu-latest
|
||||
strategy:
|
||||
matrix:
|
||||
python-version: [3.6, 3.7, 3.8]
|
||||
python-version: [3.8]
|
||||
|
||||
steps:
|
||||
- uses: actions/checkout@v2
|
||||
|
|
20
Makefile
20
Makefile
|
@ -1,10 +1,20 @@
|
|||
all: prettier mypy black
|
||||
all: format check
|
||||
|
||||
format: prettier black
|
||||
|
||||
check: mypy checkprettier checkblack
|
||||
|
||||
prettier: $(shell find lnbits -name "*.js" -name ".html")
|
||||
./node_modules/.bin/prettier --write lnbits/static/js/*.js lnbits/core/static/js/*.js lnbits/extensions/*/templates/*/*.html ./lnbits/core/templates/core/*.html lnbits/templates/*.html lnbits/extensions/*/static/js/*.js
|
||||
|
||||
mypy: $(shell find lnbits -name "*.py")
|
||||
mypy lnbits
|
||||
|
||||
black: $(shell find lnbits -name "*.py")
|
||||
black lnbits
|
||||
./venv/bin/black --line-length 120 lnbits
|
||||
|
||||
mypy: $(shell find lnbits -name "*.py")
|
||||
./venv/bin/mypy lnbits
|
||||
|
||||
checkprettier: $(shell find lnbits -name "*.js" -name ".html")
|
||||
./node_modules/.bin/prettier --check lnbits/static/js/*.js lnbits/core/static/js/*.js lnbits/extensions/*/templates/*/*.html ./lnbits/core/templates/core/*.html lnbits/templates/*.html lnbits/extensions/*/static/js/*.js
|
||||
|
||||
checkblack: $(shell find lnbits -name "*.py")
|
||||
./venv/bin/black --check --line-length 120 lnbits
|
||||
|
|
|
@ -1,8 +1,5 @@
|
|||
{
|
||||
"devDependencies": {
|
||||
"prettier": "^2.0.5"
|
||||
},
|
||||
"scripts": {
|
||||
"lint": "prettier --write lnbits/static/js/** lnbits/core/static/js/** lnbits/extensions/*/templates/**"
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue
Block a user