Rename to skip_linter

Author: VaggelisD

sqlmesh/cli/main.py

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

sqlmesh/core/context.py

@@ -1135,7 +1135,7 @@ def plan(
         no_diff: t.Optional[bool] = None,
         run: bool = False,
         diff_rendered: bool = False,
-        skip_lints: bool = False,
+        skip_linter: bool = False,
     ) -> Plan:
         """Interactively creates a plan.
 
@@ -1180,7 +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
+            skip_linter: Linter runs by default so this will skip it if enabled
 
         Returns:
             The populated Plan object.
@@ -1207,7 +1207,7 @@ def plan(
             enable_preview=enable_preview,
             run=run,
             diff_rendered=diff_rendered,
-            skip_lints=skip_lints,
+            skip_linter=skip_linter,
         )
 
         if no_auto_categorization:
@@ -1249,7 +1249,7 @@ def plan_builder(
         enable_preview: t.Optional[bool] = None,
         run: bool = False,
         diff_rendered: bool = False,
-        skip_lints: bool = False,
+        skip_linter: bool = False,
     ) -> PlanBuilder:
         """Creates a plan builder.
 
@@ -1305,7 +1305,7 @@ def plan_builder(
         if run and is_dev:
             raise ConfigError("The '--run' flag is only supported for the production environment.")
 
-        if not skip_lints:
+        if not skip_linter:
             self.lint_models(*self.models.values())
 
         self._run_plan_tests(skip_tests=skip_tests)

sqlmesh/magics.py

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

tests/cli/test_cli.py

@@ -198,7 +198,7 @@ def test_plan_skip_tests(runner, tmp_path):
     assert_backfill_success(result)
 
 
-def test_plan_skip_lints(runner, tmp_path):
+def test_plan_skip_linter(runner, tmp_path):
     create_example_project(tmp_path)
 
     with open(tmp_path / "config.yaml", "a", encoding="utf-8") as f:
@@ -212,7 +212,7 @@ def test_plan_skip_lints(runner, tmp_path):
     # 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"
+        cli, ["--log-file-dir", tmp_path, "--paths", tmp_path, "plan", "--skip-linter"], input="y\n"
     )
 
     assert result.exit_code == 0