From e1fc1566f3759700fa82d4875057192d0853b2f1 Mon Sep 17 00:00:00 2001 From: Sarah Hoffmann Date: Thu, 2 Feb 2023 09:48:20 +0100 Subject: [PATCH] fix new linting issues from pylint 2.16 --- .pylintrc | 2 +- nominatim/api/results.py | 2 +- nominatim/tools/migration.py | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) 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 -- 2.45.1