Skip to content

Commit

Permalink
Refactor unused SOURCES var in Makefile to explictly reference all so…
Browse files Browse the repository at this point in the history
…urces (#176)

This also helps if you setup a local virtual env to remove spurious results
  • Loading branch information
cogpie committed May 24, 2021
1 parent ef2faf2 commit c0b7495
Showing 1 changed file with 8 additions and 6 deletions.
14 changes: 8 additions & 6 deletions Makefile
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
SOURCES = $(shell find . -name "*.py")
SOURCE_DIRS = cfripper tests docs
SOURCE_FILES = setup.py
SOURCE_ALL = $(SOURCE_DIRS) $(SOURCE_FILES)

install:
pip install -r requirements.txt
Expand All @@ -10,19 +12,19 @@ install-docs:
pip install -e ".[dev,docs]"

format:
isort --recursive .
black .
isort --recursive $(SOURCE_ALL)
black $(SOURCE_ALL)

lint: isort-lint black-lint flake8-lint

isort-lint:
isort --check-only --recursive .
isort --check-only --recursive $(SOURCE_ALL)

black-lint:
black --check .
black --check $(SOURCE_ALL)

flake8-lint:
flake8 cfripper/ tests/
flake8 $(SOURCE_ALL)

unit:
pytest -svvv tests
Expand Down

0 comments on commit c0b7495

Please sign in to comment.