Skip to content

Commit

Permalink
make lint has a formatting check
Browse files Browse the repository at this point in the history
  • Loading branch information
epwr committed Mar 1, 2024
1 parent 324ac44 commit e212d3b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
5 changes: 1 addition & 4 deletions .github/workflows/linting.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,4 @@ jobs:
run: make venv

- name: Run Static Analysis
run: make lint

- name: Run Formatting
run: black --check app tests
run: make lint
3 changes: 2 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ VENV := venv
APP_DIR := app

# default target, when make executed without arguments
all: lint test
all: lint test format

$(VENV)/bin/activate:
python3 -m venv $(VENV)
Expand All @@ -14,6 +14,7 @@ venv: $(VENV)/bin/activate
lint: venv
./$(VENV)/bin/mypy app --strict
./$(VENV)/bin/flake8 app tests
./$(VENV)/bin/black --check app tests

# Use `yq` to parse config/testing.toml and set the env variables. Then run tests & coverage.
test: venv
Expand Down

0 comments on commit e212d3b

Please sign in to comment.