Skip to content

Commit

Permalink
Merge pull request #22 from tillywoodfield/add-linting
Browse files Browse the repository at this point in the history
Add linting
  • Loading branch information
odscjames authored Jan 30, 2024
2 parents 0edc3c1 + 9a0302e commit 81215bf
Show file tree
Hide file tree
Showing 8 changed files with 400 additions and 176 deletions.
19 changes: 19 additions & 0 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
name: Lint
on: [push, pull_request]

jobs:
lint:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-python@v5
with:
python-version: '3.10'
- name: Install requirements_dev.txt
run: pip install -r requirements_dev.txt
- name: Check isort
run: isort --check-only iatidata/
- name: Check black
run: black --check iatidata/
- name: Check flake8
run: flake8 iatidata/
9 changes: 9 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,15 @@ Running the tests:
python -m pytest iatidata/
```

Linting:

```
isort iatidata
black iatidata
flake8 iatidata
```


### Web front-end

Install Node JS 20. e.g. on Ubuntu:
Expand Down
Loading

0 comments on commit 81215bf

Please sign in to comment.