From 7988b7467b80620cd453b7b3d60b767761692cc8 Mon Sep 17 00:00:00 2001
From: jo <ljonas@riseup.net>
Date: Sat, 22 Jan 2022 16:53:36 +0100
Subject: [PATCH] test: allow to set python linters to fail per app

---
 analyzer/Makefile   | 4 ++--
 api/Makefile        | 4 ++--
 api_client/Makefile | 4 ++--
 playout/Makefile    | 4 ++--
 shared/Makefile     | 4 ++--
 tools/Makefile      | 4 ++--
 tools/python.mk     | 4 ++--
 worker/Makefile     | 4 ++--
 8 files changed, 16 insertions(+), 16 deletions(-)

diff --git a/analyzer/Makefile b/analyzer/Makefile
index 04d6253d4..48ae9f530 100644
--- a/analyzer/Makefile
+++ b/analyzer/Makefile
@@ -3,8 +3,8 @@ all: lint test
 include ../tools/python.mk
 
 PIP_INSTALL := --editable .[dev]
-PYLINT_ARG := libretime_analyzer tests
-MYPY_ARG := libretime_analyzer tests
+PYLINT_ARG := libretime_analyzer tests || true
+MYPY_ARG := libretime_analyzer tests || true
 PYTEST_ARG := --cov=libretime_analyzer tests
 
 format: .format
diff --git a/api/Makefile b/api/Makefile
index f75a1c0b0..46b0fd808 100644
--- a/api/Makefile
+++ b/api/Makefile
@@ -3,8 +3,8 @@ all: lint
 include ../tools/python.mk
 
 PIP_INSTALL := --editable .[dev]
-PYLINT_ARG := libretime_api
-MYPY_ARG := libretime_api
+PYLINT_ARG := libretime_api || true
+MYPY_ARG := libretime_api || true
 
 format: .format
 lint: .format-check .pylint .mypy
diff --git a/api_client/Makefile b/api_client/Makefile
index 0c475d8ad..b0b515a68 100644
--- a/api_client/Makefile
+++ b/api_client/Makefile
@@ -3,8 +3,8 @@ all: lint test
 include ../tools/python.mk
 
 PIP_INSTALL := --editable .
-PYLINT_ARG := libretime_api_client tests
-MYPY_ARG := libretime_api_client tests
+PYLINT_ARG := libretime_api_client tests || true
+MYPY_ARG := libretime_api_client tests || true
 PYTEST_ARG := --cov=libretime_api_client tests
 
 format: .format
diff --git a/playout/Makefile b/playout/Makefile
index b713ae99e..6f02a26f4 100644
--- a/playout/Makefile
+++ b/playout/Makefile
@@ -3,8 +3,8 @@ all: lint
 include ../tools/python.mk
 
 PIP_INSTALL := --editable .[dev]
-PYLINT_ARG := libretime_liquidsoap libretime_playout
-MYPY_ARG := libretime_liquidsoap libretime_playout
+PYLINT_ARG := libretime_liquidsoap libretime_playout || true
+MYPY_ARG := libretime_liquidsoap libretime_playout || true
 
 format: .format
 lint: .format-check .pylint .mypy
diff --git a/shared/Makefile b/shared/Makefile
index 00f9cd647..94d852b00 100644
--- a/shared/Makefile
+++ b/shared/Makefile
@@ -3,11 +3,11 @@ all: lint test
 include ../tools/python.mk
 
 PIP_INSTALL = --editable .[dev]
-PYLINT_ARG = libretime_shared tests
+PYLINT_ARG = libretime_shared tests || true
 MYPY_ARG =	--disallow-untyped-calls \
 			--disallow-untyped-defs \
 			--disallow-incomplete-defs \
-			libretime_shared
+			libretime_shared || true
 PYTEST_ARG = --cov=libretime_shared tests
 
 format: .format
diff --git a/tools/Makefile b/tools/Makefile
index 30a3814e3..961a7e117 100644
--- a/tools/Makefile
+++ b/tools/Makefile
@@ -3,8 +3,8 @@ all: lint test
 include python.mk
 
 PIP_INSTALL =
-PYLINT_ARG = tools
-MYPY_ARG = .
+PYLINT_ARG = tools || true
+MYPY_ARG = . || true
 PYTEST_ARG = .
 
 format: .format
diff --git a/tools/python.mk b/tools/python.mk
index 3ae69bdab..89d2c2ae2 100644
--- a/tools/python.mk
+++ b/tools/python.mk
@@ -43,12 +43,12 @@ install: $(VENV)
 .PHONY: .pylint
 .pylint: $(VENV)
 	source $(VENV)/bin/activate
-	pylint --jobs=$(CPU_CORES) --output-format=colorized $(PYLINT_ARG) || true
+	pylint --jobs=$(CPU_CORES) --output-format=colorized $(PYLINT_ARG)
 
 .PHONY: .mypy
 .mypy: $(VENV)
 	source $(VENV)/bin/activate
-	mypy $(MYPY_ARG) || true
+	mypy $(MYPY_ARG)
 
 .PHONY: .pytest
 .pytest: $(VENV)
diff --git a/worker/Makefile b/worker/Makefile
index d784e7e79..f3422c559 100644
--- a/worker/Makefile
+++ b/worker/Makefile
@@ -3,8 +3,8 @@ all: lint
 include ../tools/python.mk
 
 PIP_INSTALL := --editable .
-PYLINT_ARG := libretime_worker
-MYPY_ARG := libretime_worker
+PYLINT_ARG := libretime_worker || true
+MYPY_ARG := libretime_worker || true
 
 format: .format
 lint: .format-check .pylint .mypy