Commit ae847a01 authored by Niklas Bohn's avatar Niklas Bohn
Browse files

Merge branch 'enhancement/improve_linter' into 'master'

'make lint' now directly prints errors instead of only logging them to logfiles.

See merge request !80
parents 2b473fe3 e183670e
...@@ -7,7 +7,7 @@ History ...@@ -7,7 +7,7 @@ History
-------------------- --------------------
New features: New features:
* * 'make lint' now directly prints errors instead of only logging them to logfiles.
Bugfixes: Bugfixes:
* Pinned gdal to version<=3.1.2 to avoid import error. * Pinned gdal to version<=3.1.2 to avoid import error.
......
...@@ -92,10 +92,14 @@ clean-test: ## Remove test and coverage artifacts. ...@@ -92,10 +92,14 @@ clean-test: ## Remove test and coverage artifacts.
rm -rf report.html rm -rf report.html
lint: ## Check style and pep8 conformity using multiple pep8 and style checkers. For now, flake8, pycodestyle, pylint and pydocstyle are included, but their results ignored. lint: ## Check style and pep8 conformity using multiple pep8 and style checkers. For now, flake8, pycodestyle, pylint and pydocstyle are included, but their results ignored.
-flake8 --exclude="*.ipynb,*.ipynb*" --max-line-length=120 sicor tests > ./tests/linting/flake8.log -flake8 --exclude="*.ipynb,*.ipynb*" --max-line-length=120 sicor tests > ./tests/linting/flake8.log || \
-pycodestyle sicor --exclude="*.ipynb,*.ipynb*" --max-line-length=120 > ./tests/linting/pycodestyle.log (cat ./tests/linting/flake8.log && exit 1)
-pylint --rcfile=tests/linting/pylint.rc sicor > tests/linting/pylint.log -pycodestyle sicor --exclude="*.ipynb,*.ipynb*" --max-line-length=120 > ./tests/linting/pycodestyle.log || \
-pydocstyle sicor > ./tests/linting/pydocstyle.log (cat ./tests/linting/pycodestyle.log && exit 1)
-pylint --rcfile=tests/linting/pylint.rc sicor > tests/linting/pylint.log || \
(cat ./tests/linting/pylint.log && exit 1)
-pydocstyle sicor > ./tests/linting/pydocstyle.log || \
(cat ./tests/linting/pydocstyle.log && exit 1)
urlcheck: ## check for dead URLs urlcheck: ## check for dead URLs
urlchecker check . \ urlchecker check . \
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment