diff options
author | Markus Heiser <markus.heiser@darmarit.de> | 2024-03-11 14:06:26 +0100 |
---|---|---|
committer | Markus Heiser <markus.heiser@darmarIT.de> | 2024-03-11 14:55:38 +0100 |
commit | 542f7d0d7bd1a12e1884ba4a1508b40e2514d472 (patch) | |
tree | 5256282b267e494200d02914593457854567fec4 /searx/search/checker | |
parent | 8205f170ff983e5240d32dc17d7fdb526ebe5fe7 (diff) | |
download | searxng-542f7d0d7bd1a12e1884ba4a1508b40e2514d472.tar.gz searxng-542f7d0d7bd1a12e1884ba4a1508b40e2514d472.zip |
[mod] pylint all files with one profile / drop PYLINT_SEARXNG_DISABLE_OPTION
In the past, some files were tested with the standard profile, others with a
profile in which most of the messages were switched off ... some files were not
checked at all.
- ``PYLINT_SEARXNG_DISABLE_OPTION`` has been abolished
- the distinction ``# lint: pylint`` is no longer necessary
- the pylint tasks have been reduced from three to two
1. ./searx/engines -> lint engines with additional builtins
2. ./searx ./searxng_extra ./tests -> lint all other python files
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
Diffstat (limited to 'searx/search/checker')
-rw-r--r-- | searx/search/checker/__init__.py | 1 | ||||
-rw-r--r-- | searx/search/checker/__main__.py | 1 | ||||
-rw-r--r-- | searx/search/checker/background.py | 4 | ||||
-rw-r--r-- | searx/search/checker/impl.py | 12 | ||||
-rw-r--r-- | searx/search/checker/scheduler.py | 1 |
5 files changed, 8 insertions, 11 deletions
diff --git a/searx/search/checker/__init__.py b/searx/search/checker/__init__.py index 7d779a282..f6891a5db 100644 --- a/searx/search/checker/__init__.py +++ b/searx/search/checker/__init__.py @@ -1,4 +1,5 @@ # SPDX-License-Identifier: AGPL-3.0-or-later +# pylint: disable=missing-module-docstring from .impl import Checker from .background import initialize, get_result diff --git a/searx/search/checker/__main__.py b/searx/search/checker/__main__.py index 340c27187..984c8526c 100644 --- a/searx/search/checker/__main__.py +++ b/searx/search/checker/__main__.py @@ -1,5 +1,4 @@ # SPDX-License-Identifier: AGPL-3.0-or-later -# lint: pylint # pylint: disable=missing-module-docstring import sys diff --git a/searx/search/checker/background.py b/searx/search/checker/background.py index f16e4c25c..5e9d23e00 100644 --- a/searx/search/checker/background.py +++ b/searx/search/checker/background.py @@ -1,7 +1,5 @@ # SPDX-License-Identifier: AGPL-3.0-or-later -# lint: pylint -# pylint: disable=missing-module-docstring -# pyright: basic +# pylint: disable=missing-module-docstring, cyclic-import import json import time diff --git a/searx/search/checker/impl.py b/searx/search/checker/impl.py index b5dff1fd2..cf1f03449 100644 --- a/searx/search/checker/impl.py +++ b/searx/search/checker/impl.py @@ -1,4 +1,5 @@ # SPDX-License-Identifier: AGPL-3.0-or-later +# pylint: disable=missing-module-docstring, invalid-name import gc import typing @@ -149,7 +150,7 @@ def _search_query_diff( return (common, diff) -class TestResults: +class TestResults: # pylint: disable=missing-class-docstring __slots__ = 'errors', 'logs', 'languages' @@ -181,7 +182,7 @@ class TestResults: yield (test_name, error) -class ResultContainerTests: +class ResultContainerTests: # pylint: disable=missing-class-docstring __slots__ = 'test_name', 'search_query', 'result_container', 'languages', 'stop_test', 'test_results' @@ -210,7 +211,6 @@ class ResultContainerTests: if langStr: self.languages.add(langStr) self.test_results.add_language(langStr) - return None def _check_result(self, result): if not _check_no_html(result.get('title', '')): @@ -319,7 +319,7 @@ class ResultContainerTests: self._record_error(('{!r} not found in the title'.format(title))) -class CheckerTests: +class CheckerTests: # pylint: disable=missing-class-docstring, too-few-public-methods __slots__ = 'test_results', 'test_name', 'result_container_tests_list' @@ -351,7 +351,7 @@ class CheckerTests: ) -class Checker: +class Checker: # pylint: disable=missing-class-docstring __slots__ = 'processor', 'tests', 'test_results' @@ -377,7 +377,7 @@ class Checker: p.append(l) for kwargs in itertools.product(*p): - kwargs = {k: v for k, v in kwargs} + kwargs = dict(kwargs) query = kwargs['query'] params = dict(kwargs) del params['query'] diff --git a/searx/search/checker/scheduler.py b/searx/search/checker/scheduler.py index 7d7e90769..c0d3f799a 100644 --- a/searx/search/checker/scheduler.py +++ b/searx/search/checker/scheduler.py @@ -1,5 +1,4 @@ # SPDX-License-Identifier: AGPL-3.0-or-later -# lint: pylint # pylint: disable=missing-module-docstring """Lame scheduler which use Redis as a source of truth: * the Redis key SearXNG_checker_next_call_ts contains the next time the embedded checker should run. |