Set back disable too-many-public-methods

pyproject.toml

@@ -600,8 +600,7 @@ disable = [
     "consider-using-any-or-all",
     "too-many-positional-arguments",
     "unnecessary-default-type-args",
-    "logging-not-lazy",
-    "too-many-public-methods",  # TODO: Remove by someone who can bypass CI cheat linter check
+    "logging-not-lazy"
 ]
 
 # Enable the message, report, category or checker with the given id(s). You can

src/databricks/labs/ucx/assessment/workflows.py

@@ -7,7 +7,7 @@
 logger = logging.getLogger(__name__)
 
 
-class Assessment(Workflow):
+class Assessment(Workflow):  # pylint: disable=too-many-public-methods
     def __init__(self):
         super().__init__('assessment')
 

src/databricks/labs/ucx/contexts/application.py

@@ -88,6 +88,7 @@
 # used throughout the application. That being said, we'll do best
 # effort of splitting the instances between Global, Runtime,
 # Workspace CLI, and Account CLI contexts.
+# pylint: disable=too-many-public-methods
 
 logger = logging.getLogger(__name__)
 

src/databricks/labs/ucx/contexts/workflow_task.py

@@ -32,6 +32,7 @@
 from databricks.labs.ucx.progress.workflow_runs import WorkflowRunRecorder
 
 # As with GlobalContext, service factories unavoidably have a lot of public methods.
+# pylint: disable=too-many-public-methods
 
 
 class RuntimeContext(GlobalContext):

src/databricks/labs/ucx/contexts/workspace_cli.py

@@ -29,6 +29,8 @@
 
 logger = logging.getLogger(__name__)
 
+# pylint: disable=too-many-public-methods
+
 
 class WorkspaceContext(CliContext):
     def __init__(self, ws: WorkspaceClient, named_parameters: dict[str, str] | None = None):

src/databricks/labs/ucx/hive_metastore/tables.py

@@ -48,7 +48,7 @@ class AclMigrationWhat(Enum):
 
 
 @dataclass
-class Table:
+class Table:  # pylint: disable=too-many-public-methods
     catalog: str
     database: str
     name: str

src/databricks/labs/ucx/source_code/python/python_ast.py

@@ -68,7 +68,7 @@ def first_statement(self) -> NodeNG | None:
         return self.tree.first_statement()
 
 
-class Tree:
+class Tree:  # pylint: disable=too-many-public-methods
 
     @classmethod
     def maybe_parse(cls, code: str) -> MaybeTree:

tests/integration/conftest.py

@@ -453,7 +453,9 @@ def workspace_client(self) -> WorkspaceClient:
         return self._ws
 
 
-class MockRuntimeContext(CommonUtils, RuntimeContext):  # pylint: disable=too-many-instance-attributes
+class MockRuntimeContext(
+    CommonUtils, RuntimeContext
+):  # pylint: disable=too-many-instance-attributes,too-many-public-methods
     def __init__(  # pylint: disable=too-many-arguments
         self,
         make_catalog_fixture,