From: Sarah Hoffmann Date: Thu, 2 Feb 2023 08:48:20 +0000 (+0100) Subject: fix new linting issues from pylint 2.16 X-Git-Tag: v4.3.0~106^2~8 X-Git-Url: https://git.openstreetmap.org./nominatim.git/commitdiff_plain/e1fc1566f3759700fa82d4875057192d0853b2f1 fix new linting issues from pylint 2.16 --- diff --git a/.pylintrc b/.pylintrc index b230c4ec..cbb26a4e 100644 --- a/.pylintrc +++ b/.pylintrc @@ -13,6 +13,6 @@ ignored-classes=NominatimArgs,closing # 'too-many-ancestors' is triggered already by deriving from UserDict # 'not-context-manager' disabled because it causes false positives once # typed Python is enabled. See also https://github.com/PyCQA/pylint/issues/5273 -disable=too-few-public-methods,duplicate-code,too-many-ancestors,bad-option-value,no-self-use,not-context-manager +disable=too-few-public-methods,duplicate-code,too-many-ancestors,bad-option-value,no-self-use,not-context-manager,use-dict-literal good-names=i,x,y,m,t,fd,db,cc diff --git a/nominatim/api/results.py b/nominatim/api/results.py index b952f6fd..779911e9 100644 --- a/nominatim/api/results.py +++ b/nominatim/api/results.py @@ -11,7 +11,7 @@ Data classes are part of the public API while the functions are for internal use only. That's why they are implemented as free-standing functions instead of member functions. """ -from typing import Optional, Tuple, Dict, Sequence, Any +from typing import Optional, Tuple, Dict, Sequence import enum import dataclasses import datetime as dt diff --git a/nominatim/tools/migration.py b/nominatim/tools/migration.py index 545f5c48..7d117a8c 100644 --- a/nominatim/tools/migration.py +++ b/nominatim/tools/migration.py @@ -104,7 +104,7 @@ def _migration(major: int, minor: int, patch: int = 0, there. """ def decorator(func: Callable[..., None]) -> Callable[..., None]: - version = (NominatimVersion(major, minor, patch, dbpatch)) + version = NominatimVersion(major, minor, patch, dbpatch) _MIGRATION_FUNCTIONS.append((version, func)) return func