Add skip_lints

Author: VaggelisD

sqlmesh/cli/main.py

@@ -339,6 +339,11 @@ def diff(ctx: click.Context, environment: t.Optional[str] = None) -> None:
     is_flag=True,
     help="Skip tests prior to generating the plan if they are defined.",
 )
+@click.option(
+    "--skip-lints",
+    is_flag=True,
+    help="Skip linting prior to generating the plan if the linter is enabled.",
+)
 @click.option(
     "--restate-model",
     "-r",

sqlmesh/core/context.py

@@ -1135,6 +1135,7 @@ def plan(
         no_diff: t.Optional[bool] = None,
         run: bool = False,
         diff_rendered: bool = False,
+        skip_lints: bool = False,
     ) -> Plan:
         """Interactively creates a plan.
 
@@ -1179,6 +1180,7 @@ def plan(
             no_diff: Hide text differences for changed models.
             run: Whether to run latest intervals as part of the plan application.
             diff_rendered: Whether the diff should compare raw vs rendered models
+            skip_lints: Linter runs by default so this will skip it if enabled
 
         Returns:
             The populated Plan object.
@@ -1205,6 +1207,7 @@ def plan(
             enable_preview=enable_preview,
             run=run,
             diff_rendered=diff_rendered,
+            skip_lints=skip_lints,
         )
 
         if no_auto_categorization:
@@ -1246,6 +1249,7 @@ def plan_builder(
         enable_preview: t.Optional[bool] = None,
         run: bool = False,
         diff_rendered: bool = False,
+        skip_lints: bool = False,
     ) -> PlanBuilder:
         """Creates a plan builder.
 
@@ -1301,7 +1305,8 @@ def plan_builder(
         if run and is_dev:
             raise ConfigError("The '--run' flag is only supported for the production environment.")
 
-        self.lint_models(*self.models.values())
+        if not skip_lints:
+            self.lint_models(*self.models.values())
 
         self._run_plan_tests(skip_tests=skip_tests)
 

sqlmesh/magics.py

@@ -343,6 +343,11 @@ def test(self, context: Context, line: str, test_def_raw: t.Optional[str] = None
         action="store_true",
         help="Skip the unit tests defined for the model.",
     )
+    @argument(
+        "--skip-lints",
+        action="store_true",
+        help="Skip the linter for the model.",
+    )
     @argument(
         "--restate-model",
         "-r",

tests/cli/test_cli.py

@@ -198,6 +198,29 @@ def test_plan_skip_tests(runner, tmp_path):
     assert_backfill_success(result)
 
 
+def test_plan_skip_lints(runner, tmp_path):
+    create_example_project(tmp_path)
+
+    with open(tmp_path / "config.yaml", "a", encoding="utf-8") as f:
+        f.write(
+            """linter:
+        enabled: True
+        rules: "ALL"
+    """
+        )
+
+    # Successful test run message should not appear with `--skip-tests`
+    # Input: `y` to apply and backfill
+    result = runner.invoke(
+        cli, ["--log-file-dir", tmp_path, "--paths", tmp_path, "plan", "--skip-lints"], input="y\n"
+    )
+
+    assert result.exit_code == 0
+    assert "Linter warnings" not in result.output
+    assert_new_env(result)
+    assert_backfill_success(result)
+
+
 def test_plan_restate_model(runner, tmp_path):
     create_example_project(tmp_path)
     init_prod_and_backfill(runner, tmp_path)