mirror of https://github.com/apache/superset.git
chore(pylint): Reenable too-few-public-methods check (#16264)
Co-authored-by: John Bodley <john.bodley@airbnb.com>
This commit is contained in:
parent
0df15bf207
commit
36bc7b0b80
|
@ -85,7 +85,6 @@ disable=
|
||||||
missing-docstring,
|
missing-docstring,
|
||||||
too-many-lines,
|
too-many-lines,
|
||||||
raise-missing-from,
|
raise-missing-from,
|
||||||
too-few-public-methods,
|
|
||||||
duplicate-code,
|
duplicate-code,
|
||||||
|
|
||||||
[REPORTS]
|
[REPORTS]
|
||||||
|
|
|
@ -43,7 +43,7 @@ class BaseCommand(ABC):
|
||||||
"""
|
"""
|
||||||
|
|
||||||
|
|
||||||
class CreateMixin:
|
class CreateMixin: # pylint: disable=too-few-public-methods
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def populate_owners(
|
def populate_owners(
|
||||||
user: User, owner_ids: Optional[List[int]] = None
|
user: User, owner_ids: Optional[List[int]] = None
|
||||||
|
@ -61,7 +61,7 @@ class CreateMixin:
|
||||||
return populate_owners(user, owner_ids, default_to_user=True)
|
return populate_owners(user, owner_ids, default_to_user=True)
|
||||||
|
|
||||||
|
|
||||||
class UpdateMixin:
|
class UpdateMixin: # pylint: disable=too-few-public-methods
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def populate_owners(
|
def populate_owners(
|
||||||
user: User, owner_ids: Optional[List[int]] = None
|
user: User, owner_ids: Optional[List[int]] = None
|
||||||
|
|
|
@ -214,7 +214,7 @@ def extra_validator(value: str) -> str:
|
||||||
return value
|
return value
|
||||||
|
|
||||||
|
|
||||||
class DatabaseParametersSchemaMixin:
|
class DatabaseParametersSchemaMixin: # pylint: disable=too-few-public-methods
|
||||||
"""
|
"""
|
||||||
Allow SQLAlchemy URI to be passed as separate parameters.
|
Allow SQLAlchemy URI to be passed as separate parameters.
|
||||||
|
|
||||||
|
|
|
@ -98,7 +98,7 @@ class UIManifestProcessor:
|
||||||
return self.manifest.get(bundle, {}).get(asset_type, [])
|
return self.manifest.get(bundle, {}).get(asset_type, [])
|
||||||
|
|
||||||
|
|
||||||
class ProfilingExtension:
|
class ProfilingExtension: # pylint: disable=too-few-public-methods
|
||||||
def __init__(self, interval: float = 1e-4) -> None:
|
def __init__(self, interval: float = 1e-4) -> None:
|
||||||
self.interval = interval
|
self.interval = interval
|
||||||
|
|
||||||
|
|
|
@ -22,7 +22,7 @@ from sqlalchemy import TypeDecorator
|
||||||
from sqlalchemy_utils import EncryptedType
|
from sqlalchemy_utils import EncryptedType
|
||||||
|
|
||||||
|
|
||||||
class AbstractEncryptedFieldAdapter(ABC):
|
class AbstractEncryptedFieldAdapter(ABC): # pylint: disable=too-few-public-methods
|
||||||
@abstractmethod
|
@abstractmethod
|
||||||
def create(
|
def create(
|
||||||
self,
|
self,
|
||||||
|
@ -33,7 +33,9 @@ class AbstractEncryptedFieldAdapter(ABC):
|
||||||
pass
|
pass
|
||||||
|
|
||||||
|
|
||||||
class SQLAlchemyUtilsAdapter(AbstractEncryptedFieldAdapter):
|
class SQLAlchemyUtilsAdapter( # pylint: disable=too-few-public-methods
|
||||||
|
AbstractEncryptedFieldAdapter
|
||||||
|
):
|
||||||
def create(
|
def create(
|
||||||
self,
|
self,
|
||||||
app_config: Optional[Dict[str, Any]],
|
app_config: Optional[Dict[str, Any]],
|
||||||
|
|
|
@ -27,7 +27,7 @@ except ModuleNotFoundError:
|
||||||
Profiler = None
|
Profiler = None
|
||||||
|
|
||||||
|
|
||||||
class SupersetProfiler:
|
class SupersetProfiler: # pylint: disable=too-few-public-methods
|
||||||
"""
|
"""
|
||||||
WSGI middleware to instrument Superset.
|
WSGI middleware to instrument Superset.
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue