refactor: breaking up the console interface

Author: benfdking

sqlmesh/core/console.py

@@ -83,7 +83,15 @@
 RED_X_MARK = "\u274c"
 
 
-class Console(abc.ABC):
+class LinterConsole(abc.ABC):
+    @abc.abstractmethod
+    def show_linter_violations(
+        self, violations: t.List[RuleViolation], model: Model, is_error: bool = False
+    ) -> None:
+        """Prints all linter violations depending on their severity"""
+
+
+class Console(LinterConsole, abc.ABC):
     """Abstract base class for defining classes used for displaying information to the user and also interact
     with them when their input is needed."""
 
@@ -400,12 +408,6 @@ def _limit_model_names(self, tree: Tree, verbosity: Verbosity = Verbosity.DEFAUL
             ]
         return tree
 
-    @abc.abstractmethod
-    def show_linter_violations(
-        self, violations: t.List[RuleViolation], model: Model, is_error: bool = False
-    ) -> None:
-        """Prints all linter violations depending on their severity"""
-
 
 class NoopConsole(Console):
     def start_plan_evaluation(self, plan: EvaluatablePlan) -> None:

sqlmesh/core/linter/definition.py

@@ -7,7 +7,7 @@
 from sqlmesh.core.model import Model
 
 from sqlmesh.utils.errors import raise_config_error
-from sqlmesh.core.console import get_console, Console
+from sqlmesh.core.console import LinterConsole, get_console, Console
 from sqlmesh.core.linter.rule import RuleSet
 
 
@@ -50,7 +50,7 @@ def from_rules(cls, all_rules: RuleSet, config: LinterConfig) -> Linter:
 
         return Linter(config.enabled, all_rules, rules, warn_rules)
 
-    def lint_model(self, model: Model, console: Console = get_console()) -> bool:
+    def lint_model(self, model: Model, console: LinterConsole = get_console()) -> bool:
         if not self.enabled:
             return False