Rename to formatting

Author: VaggelisD

docs/concepts/models/overview.md

@@ -457,6 +457,9 @@ to `false` causes SQLMesh to disable query canonicalization & simplification. Th
 ### ignored_rules
 : Specifies which linter rules should be ignored/excluded for this model.
 
+### formatting
+:   Whether the model will be formatted. All models are formatted by default. Setting this to `false` causes SQLMesh to ignore this model during `sqlmesh format`. 
+
 ## Incremental Model Properties
 
 These properties can be specified in an incremental model's `kind` definition.

docs/reference/model_configuration.md

@@ -40,7 +40,7 @@ Configuration options for SQLMesh model properties. Supported by all model kinds
 | `gateway`             | Specifies the gateway to use for the execution of this model. When not specified, the default gateway is used.                                                                                                                                                                                                       |       str        |    N     |
 | `optimize_query`             | Whether the model's query should be optimized. This attribute is `true` by default. Setting it to `false` causes SQLMesh to disable query canonicalization & simplification. This should be turned off only if the optimized query leads to errors such as surpassing text limit.                                                                                                                                                                                                      |       bool        |    N     |
 | `ignored_rules`             |  A list of linter rule names (or "ALL") to be ignored/excluded for this model                                                                                                                                                                                             |       str \| array[str]        |    N     |
-
+| `formatting`             | Whether the model will be formatted. All models are formatted by default. Setting this to `false` causes SQLMesh to ignore this model during `sqlmesh format`.                                                                                                                                                                                                      |       bool        |    N     |
 ### Model defaults
 
 The SQLMesh project-level configuration must contain the `model_defaults` key and must specify a value for its `dialect` key. Other values are set automatically unless explicitly overridden in the model definition. Learn more about project-level configuration in the [configuration guide](../guides/configuration.md).

sqlmesh/core/audit/definition.py

@@ -71,7 +71,7 @@ class AuditMixin(AuditCommonMetaMixin):
     defaults: t.Dict[str, exp.Expression]
     expressions_: t.Optional[t.List[exp.Expression]]
     jinja_macros: JinjaMacroRegistry
-    format: t.Optional[bool] = Field(default=None, exclude=True)
+    formatting: t.Optional[bool] = Field(default=None, exclude=True)
 
     @property
     def expressions(self) -> t.List[exp.Expression]:

sqlmesh/core/config/model.py

@@ -60,7 +60,7 @@ class ModelDefaultsConfig(BaseConfig):
     allow_partials: t.Optional[t.Union[str, bool]] = None
     interval_unit: t.Optional[t.Union[str, IntervalUnit]] = None
     enabled: t.Optional[t.Union[str, bool]] = None
-    format: t.Optional[t.Union[str, bool]] = None
+    formatting: t.Optional[t.Union[str, bool]] = None
 
     _model_kind_validator = model_kind_validator
     _on_destructive_change_validator = on_destructive_change_validator

sqlmesh/core/context.py

@@ -1089,7 +1089,7 @@ def format(
 
         for target in filtered_targets:
             if (
-                target._path is None or target.format is False
+                target._path is None or target.formatting is False
             ):  # introduced to satisfy type checker as still want to pull filter out as many targets as possible before loop
                 continue
 

sqlmesh/core/model/common.py

@@ -386,7 +386,7 @@ def sorted_python_env_payloads(python_env: t.Dict[str, Executable]) -> t.List[st
     "allow_partials",
     "enabled",
     "optimize_query",
-    "format",
+    "formatting",
     mode="before",
     check_fields=False,
 )(parse_bool)

sqlmesh/core/model/meta.py

@@ -80,7 +80,7 @@ class ModelMeta(_Node):
     ignored_rules_: t.Optional[t.Set[str]] = Field(
         default=None, exclude=True, alias="ignored_rules"
     )
-    format: t.Optional[bool] = Field(default=None, exclude=True)
+    formatting: t.Optional[bool] = Field(default=None, exclude=True)
 
     _bool_validator = bool_validator
     _model_kind_validator = model_kind_validator

tests/core/test_audit.py

@@ -962,13 +962,13 @@ def test_multiple_audits_with_same_name():
     assert model.audits[1][1] == model.audits[2][1]
 
 
-def test_audit_format_flag_serde():
+def test_audit_formatting_flag_serde():
     expressions = parse(
         """
         AUDIT (
             name my_audit,
             dialect bigquery,
-            format false,
+            formatting false,
         );
 
         SELECT * FROM db.table WHERE col = @VAR('test_var')
@@ -984,7 +984,7 @@ def test_audit_format_flag_serde():
 
     audit_json = audit.json()
 
-    assert "format" not in json.loads(audit_json)
+    assert "formatting" not in json.loads(audit_json)
 
     deserialized_audit = ModelAudit.parse_raw(audit_json)
     assert deserialized_audit.dict() == audit.dict()

tests/core/test_format.py

@@ -103,21 +103,23 @@ def test_format_files(tmp_path: pathlib.Path, mocker: MockerFixture):
     )
 
 
-def test_ignore_format_files(tmp_path: pathlib.Path):
+def test_ignore_formating_files(tmp_path: pathlib.Path):
     models_dir = pathlib.Path("models")
     audits_dir = pathlib.Path("audits")
 
     # Case 1: Model and Audit are not formatted if the flag is set to false (overriding defaults)
-    model1_text = "MODEL(name this.model1, dialect 'duckdb', format false); SELECT 1 col"
+    model1_text = "MODEL(name this.model1, dialect 'duckdb', formatting false); SELECT 1 col"
     model1 = create_temp_file(tmp_path, pathlib.Path(models_dir, "model_1.sql"), model1_text)
 
-    audit1_text = "AUDIT(name audit1, dialect 'duckdb', format false); SELECT col1 col2 FROM @this_model WHERE     foo < 0;"
+    audit1_text = "AUDIT(name audit1, dialect 'duckdb', formatting false); SELECT col1 col2 FROM @this_model WHERE     foo < 0;"
     audit1 = create_temp_file(tmp_path, pathlib.Path(audits_dir, "audit_1.sql"), audit1_text)
 
-    audit2_text = "AUDIT(name audit2, dialect 'duckdb', standalone true, format false); SELECT col1 col2 FROM @this_model WHERE     foo < 0;"
+    audit2_text = "AUDIT(name audit2, dialect 'duckdb', standalone true, formatting false); SELECT col1 col2 FROM @this_model WHERE     foo < 0;"
     audit2 = create_temp_file(tmp_path, pathlib.Path(audits_dir, "audit_2.sql"), audit2_text)
 
-    Context(paths=tmp_path, config=Config(model_defaults=ModelDefaultsConfig(format=True))).format()
+    Context(
+        paths=tmp_path, config=Config(model_defaults=ModelDefaultsConfig(formatting=True))
+    ).format()
 
     assert model1.read_text(encoding="utf-8") == model1_text
     assert audit1.read_text(encoding="utf-8") == audit1_text
@@ -127,20 +129,19 @@ def test_ignore_format_files(tmp_path: pathlib.Path):
     model2_text = "MODEL(name this.model2, dialect 'duckdb'); SELECT 1 col"
     model2 = create_temp_file(tmp_path, pathlib.Path(models_dir, "model_2.sql"), model2_text)
 
-    model3_text = (
-        "MODEL(name this.model3, dialect 'duckdb', format false, format true); SELECT 1 col"
-    )
+    model3_text = "MODEL(name this.model3, dialect 'duckdb', formatting true); SELECT 1 col"
     model3 = create_temp_file(tmp_path, pathlib.Path(models_dir, "model_3.sql"), model3_text)
 
     Context(
-        paths=tmp_path, config=Config(model_defaults=ModelDefaultsConfig(format=False))
+        paths=tmp_path, config=Config(model_defaults=ModelDefaultsConfig(formatting=False))
     ).format()
 
-    # Model is not formatted if the defaults flag is set to false
+    # Case 2.1: Model is not formatted if the defaults flag is set to false
     assert model2.read_text(encoding="utf-8") == model2_text
 
-    # Model is formatted if it's flag is set to true, overriding defaults
+    print(f"model3: {model3.read_text(encoding='utf-8')}")
+    # Case 2.2: Model is formatted if it's flag is set to true, overriding defaults
     assert (
         model3.read_text(encoding="utf-8")
-        == "MODEL (\n  name this.model3,\n  dialect 'duckdb',\n  format FALSE,\n  format TRUE\n);\n\nSELECT\n  1 AS col"
+        == "MODEL (\n  name this.model3,\n  dialect 'duckdb',\n  formatting TRUE\n);\n\nSELECT\n  1 AS col"
     )

tests/core/test_model.py

@@ -9031,12 +9031,12 @@ def test_var_in_def(assert_exp_eq):
     )
 
 
-def test_format_flag_serde():
+def test_formatting_flag_serde():
     expressions = d.parse(
         """
         MODEL(
             name test_model,
-            format False,
+            formatting False,
         );
         SELECT * FROM tbl;
     """,
@@ -9047,7 +9047,7 @@ def test_format_flag_serde():
 
     model_json = model.json()
 
-    assert "format" not in json.loads(model_json)
+    assert "formatting" not in json.loads(model_json)
 
     deserialized_model = SqlModel.parse_raw(model_json)
     assert deserialized_model.dict() == model.dict()