Change ruff line-length setting to 120 (#3675)

## Changes

This changes the ruff formatter line-length from 150 to 120.

## Why 120

* PEP8 line length is 79 ← _this is extremely tight_
* Black and Ruff default line length is 88 ← _this is good since it's a
tooling default but, still rather tight_
* Current line length was 150 to balance line length vs. spreading code
over many lines
* The [workspace
formatter](https://docs.databricks.com/aws/en/notebooks/notebooks-code#python-black-formatter-library)
defaults to 120 ← _this seems like a sweet spot for us; it means our
templates by default use the same setting as the workspace_

Note that for IDE support of our templates, we could set a `tool.black`
section as done in
https://github.com/databricks/cli/pull/3671/files?w=1#diff-ed1a9755eebbb3b630edc93a5731f9dd8caa66b9967f9a57234c6ce9bde88c33R25-R26.
Doing that is out of scope for the present PR.

## Tests
Standard acceptance tests.

---------

Co-authored-by: Claude <[email protected]>

Author: lennartkats-db

acceptance/bin/yamlcheck.py

@@ -33,9 +33,21 @@ def main():
         if original_content != formatted_content:
             has_changes.append(str(yaml_file))
             # Add $ markers for trailing whitespace
-            original_with_markers = [line.rstrip("\n") + ("$" if line.rstrip() != line.rstrip("\n") else "") + "\n" for line in original_content]
-            formatted_with_markers = [line.rstrip("\n") + ("$" if line.rstrip() != line.rstrip("\n") else "") + "\n" for line in formatted_content]
-            diff = unified_diff(original_with_markers, formatted_with_markers, fromfile=str(yaml_file), tofile=str(yaml_file), lineterm="")
+            original_with_markers = [
+                line.rstrip("\n") + ("$" if line.rstrip() != line.rstrip("\n") else "") + "\n"
+                for line in original_content
+            ]
+            formatted_with_markers = [
+                line.rstrip("\n") + ("$" if line.rstrip() != line.rstrip("\n") else "") + "\n"
+                for line in formatted_content
+            ]
+            diff = unified_diff(
+                original_with_markers,
+                formatted_with_markers,
+                fromfile=str(yaml_file),
+                tofile=str(yaml_file),
+                lineterm="",
+            )
             print("".join(diff))
 
     if has_changes:

acceptance/bundle/templates/default-python/classic/output/my_default_python/tests/conftest.py

@@ -10,7 +10,9 @@
     from pyspark.sql import SparkSession
     import pytest
 except ImportError:
-    raise ImportError("Test dependencies not found.\n\nRun tests using 'uv run pytest'. See http://docs.astral.sh/uv to learn more about uv.")
+    raise ImportError(
+        "Test dependencies not found.\n\nRun tests using 'uv run pytest'. See http://docs.astral.sh/uv to learn more about uv."
+    )
 
 
 def enable_fallback_compute():

acceptance/bundle/templates/default-python/serverless/output/my_default_python/tests/conftest.py

@@ -10,7 +10,9 @@
     from pyspark.sql import SparkSession
     import pytest
 except ImportError:
-    raise ImportError("Test dependencies not found.\n\nRun tests using 'uv run pytest'. See http://docs.astral.sh/uv to learn more about uv.")
+    raise ImportError(
+        "Test dependencies not found.\n\nRun tests using 'uv run pytest'. See http://docs.astral.sh/uv to learn more about uv."
+    )
 
 
 def enable_fallback_compute():

acceptance/bundle/templates/lakeflow-pipelines/python/output/my_lakeflow_pipelines/resources/lakeflow_pipelines_etl/transformations/sample_zones_my_lakeflow_pipelines.py

@@ -10,4 +10,8 @@
 @dp.table
 def sample_zones_my_lakeflow_pipelines():
     # Read from the "sample_trips" table, then sum all the fares
-    return spark.read.table(f"sample_trips_my_lakeflow_pipelines").groupBy(col("pickup_zip")).agg(sum("fare_amount").alias("total_fare"))
+    return (
+        spark.read.table(f"sample_trips_my_lakeflow_pipelines")
+        .groupBy(col("pickup_zip"))
+        .agg(sum("fare_amount").alias("total_fare"))
+    )

acceptance/install_terraform.py

@@ -68,9 +68,7 @@ def main():
         terraform_provider_version = args.provider_version
 
     terraform_provider_file = f"terraform-provider-databricks_{terraform_provider_version}_{os_name}_{arch}.zip"
-    terraform_provider_url = (
-        f"https://github.com/databricks/terraform-provider-databricks/releases/download/v{terraform_provider_version}/{terraform_provider_file}"
-    )
+    terraform_provider_url = f"https://github.com/databricks/terraform-provider-databricks/releases/download/v{terraform_provider_version}/{terraform_provider_file}"
 
     target.mkdir(exist_ok=True, parents=True)
 
@@ -90,7 +88,9 @@ def main():
         terraform_path.chmod(0o755)
 
     tfplugins_path = target / "tfplugins"
-    provider_dir = Path(tfplugins_path / f"registry.terraform.io/databricks/databricks/{terraform_provider_version}/{os_name}_{arch}")
+    provider_dir = Path(
+        tfplugins_path / f"registry.terraform.io/databricks/databricks/{terraform_provider_version}/{os_name}_{arch}"
+    )
     if not provider_dir.exists():
         print(f"Extracting {terraform_provider_path} -> {provider_dir}")
         os.makedirs(provider_dir, exist_ok=True)

acceptance/pipelines/e2e/output/my_project/transformations/sample_trips_my_project.py

@@ -10,4 +10,6 @@
 
 @dlt.table
 def sample_trips_my_project():
-    return spark.read.table("samples.nyctaxi.trips").withColumn("trip_distance_km", utils.distance_km(col("trip_distance")))
+    return spark.read.table("samples.nyctaxi.trips").withColumn(
+        "trip_distance_km", utils.distance_km(col("trip_distance"))
+    )

acceptance/pipelines/e2e/output/my_project/transformations/sample_zones_my_project.py

@@ -10,4 +10,8 @@
 @dlt.table
 def sample_zones_my_project():
     # Read from the "sample_trips" table, then sum all the fares
-    return spark.read.table(f"sample_trips_my_project").groupBy(col("pickup_zip")).agg(sum("fare_amount").alias("total_fare"))
+    return (
+        spark.read.table(f"sample_trips_my_project")
+        .groupBy(col("pickup_zip"))
+        .agg(sum("fare_amount").alias("total_fare"))
+    )

acceptance/pipelines/init/error-cases/output/my_project/transformations/sample_trips_my_project.py

@@ -10,4 +10,6 @@
 
 @dlt.table
 def sample_trips_my_project():
-    return spark.read.table("samples.nyctaxi.trips").withColumn("trip_distance_km", utils.distance_km(col("trip_distance")))
+    return spark.read.table("samples.nyctaxi.trips").withColumn(
+        "trip_distance_km", utils.distance_km(col("trip_distance"))
+    )

acceptance/pipelines/init/error-cases/output/my_project/transformations/sample_zones_my_project.py

@@ -10,4 +10,8 @@
 @dlt.table
 def sample_zones_my_project():
     # Read from the "sample_trips" table, then sum all the fares
-    return spark.read.table(f"sample_trips_my_project").groupBy(col("pickup_zip")).agg(sum("fare_amount").alias("total_fare"))
+    return (
+        spark.read.table(f"sample_trips_my_project")
+        .groupBy(col("pickup_zip"))
+        .agg(sum("fare_amount").alias("total_fare"))
+    )

acceptance/pipelines/init/python/output/my_python_project/transformations/sample_trips_my_python_project.py

@@ -10,4 +10,6 @@
 
 @dlt.table
 def sample_trips_my_python_project():
-    return spark.read.table("samples.nyctaxi.trips").withColumn("trip_distance_km", utils.distance_km(col("trip_distance")))
+    return spark.read.table("samples.nyctaxi.trips").withColumn(
+        "trip_distance_km", utils.distance_km(col("trip_distance"))
+    )