From 6b0a896f01eb0c59a5ce8d46c18b3549f0007901 Mon Sep 17 00:00:00 2001 From: Markus Heiser Date: Sun, 22 Nov 2020 11:37:12 +0100 Subject: [PATCH] [mod] digg - pylint searx/engines/digg.py Eliminate redundant file names which are tested by test.pylint and ignored by test.pep8 Signed-off-by: Markus Heiser --- Makefile | 16 ++++++++-------- searx/engines/digg.py | 1 + 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/Makefile b/Makefile index 326c19336..4a873f0c6 100644 --- a/Makefile +++ b/Makefile @@ -212,15 +212,15 @@ gecko.driver: PHONY += test test.sh test.pylint test.pep8 test.unit test.coverage test.robot test: buildenv test.pylint test.pep8 test.unit gecko.driver test.robot +PYLINT_FILES=\ + searx/preferences.py \ + searx/testing.py \ + searx/engines/gigablast.py \ + searx/engines/deviantart.py \ + searx/engines/digg.py -# TODO: balance linting with pylint test.pylint: pyenvinstall - $(call cmd,pylint,\ - searx/preferences.py \ - searx/testing.py \ - searx/engines/gigablast.py \ - searx/engines/deviantart.py \ - ) + $(call cmd,pylint,$(PYLINT_FILES)) $(call cmd,pylint,\ --disable=$(PYLINT_SEARX_DISABLE_OPTION) \ --additional-builtins=$(PYLINT_ADDITIONAL_BUILTINS_FOR_ENGINES) \ @@ -249,7 +249,7 @@ test.sh: test.pep8: pyenvinstall @echo "TEST pycodestyle (formerly pep8)" - $(Q)$(PY_ENV_ACT); pycodestyle --exclude='searx/static, searx/languages.py, searx/engines/gigablast.py, searx/engines/deviantart.py' \ + $(Q)$(PY_ENV_ACT); pycodestyle --exclude='searx/static, searx/languages.py, $(foreach f,$(PYLINT_FILES),$(f),)' \ --max-line-length=120 --ignore "E117,E252,E402,E722,E741,W503,W504,W605" searx tests test.unit: pyenvinstall diff --git a/searx/engines/digg.py b/searx/engines/digg.py index cb007d40d..63bce51cf 100644 --- a/searx/engines/digg.py +++ b/searx/engines/digg.py @@ -9,6 +9,7 @@ @stable no (HTML can change) @parse url, title, content, publishedDate, thumbnail """ +# pylint: disable=missing-function-docstring import random import string