aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel Hahler <git@thequod.de>2019-08-08 16:02:59 +0200
committerGitHub <noreply@github.com>2019-08-08 16:02:59 +0200
commit0288ba6e3820ffa05fbb435a678cbc66e1ac730d (patch)
tree4f97f7317f784c1d139ba5453f7cc22b9915e7f7
parent38a3af5dffd072e465fddbdc74ca816a71b9ee31 (diff)
downloadrneovim-0288ba6e3820ffa05fbb435a678cbc66e1ac730d.tar.gz
rneovim-0288ba6e3820ffa05fbb435a678cbc66e1ac730d.tar.bz2
rneovim-0288ba6e3820ffa05fbb435a678cbc66e1ac730d.zip
build: Makefile: use _opt_pylint [ci skip] (#10720)
A separate rule it clearer, and allows for `make --keep-going lint` in general later. Ref: https://github.com/neovim/neovim/pull/10714
-rw-r--r--Makefile11
1 files changed, 6 insertions, 5 deletions
diff --git a/Makefile b/Makefile
index 97bb3f43ae..fff0ffef28 100644
--- a/Makefile
+++ b/Makefile
@@ -141,6 +141,11 @@ lualint: | build/.ran-cmake deps
pylint:
flake8 contrib/ scripts/ src/ test/
+# Run pylint only if flake8 is installed.
+_opt_pylint:
+ @command -v flake8 && { $(MAKE) pylint; exit $$?; } \
+ || echo "SKIP: pylint (flake8 not found)"
+
unittest: | nvim
+$(BUILD_CMD) -C build unittest
@@ -182,11 +187,7 @@ appimage:
appimage-%:
bash scripts/genappimage.sh $*
-lint: check-single-includes clint lualint
- @# Run pylint only if flake8 is installed.
- @command -v flake8 \
- && { $(MAKE) pylint; exit $$?; } \
- || echo "SKIP: pylint (flake8 not found)"
+lint: check-single-includes clint lualint _opt_pylint
# Generic pattern rules, allowing for `make build/bin/nvim` etc.
# Does not work with "Unix Makefiles".