[ruff] 0.1.1 -> 0.1.7 (#18531)

## Summary & Motivation

Internal companion PR: dagster-io/internal#7638

Bump ruff from 0.1.1 to 0.1.7. This fixes bugs, has a few performance
improvements, and takes the formatter out of alpha.

The vast majority of changes are from a change to the formatter that
adds a trailing comma in multiline parameter specifications.

## How I Tested These Changes

`make ruff`

Author: smackesey

.pre-commit-config.yaml

@@ -1,6 +1,6 @@
 repos:
 - repo: https://github.com/charliermarsh/ruff-pre-commit
-  rev: v0.1.1
+  rev: v0.1.7
   hooks:
     - id: ruff
       args: [--fix, --exit-non-zero-on-fix]

docs/content/concepts/assets/asset-checks.mdx

@@ -192,7 +192,7 @@ def items():
 
 
 def make_checks(
-    check_blobs: Sequence[Mapping[str, str]]
+    check_blobs: Sequence[Mapping[str, str]],
 ) -> Sequence[AssetChecksDefinition]:
     checks = []
     for check_blob in check_blobs:

docs/content/integrations/deltalake/reference.mdx

@@ -191,7 +191,7 @@ from dagster import (
     metadata={"partition_expr": {"date": "time", "species": "species"}},
 )
 def iris_dataset_partitioned(context) -> pd.DataFrame:
-    partition = partition = context.partition_key.keys_by_dimension
+    partition = context.partition_key.keys_by_dimension
     species = partition["species"]
     date = partition["date"]
 

examples/docs_snippets/docs_snippets/concepts/assets/asset_checks/factory.py

@@ -20,7 +20,7 @@ def items():
 
 
 def make_checks(
-    check_blobs: Sequence[Mapping[str, str]]
+    check_blobs: Sequence[Mapping[str, str]],
 ) -> Sequence[AssetChecksDefinition]:
     checks = []
     for check_blob in check_blobs:

examples/docs_snippets/docs_snippets/integrations/deltalake/multi_partition.py

@@ -26,7 +26,7 @@ def get_iris_data_for_date(*args, **kwargs):
     metadata={"partition_expr": {"date": "time", "species": "species"}},
 )
 def iris_dataset_partitioned(context) -> pd.DataFrame:
-    partition = partition = context.partition_key.keys_by_dimension
+    partition = context.partition_key.keys_by_dimension
     species = partition["species"]
     date = partition["date"]
 

helm/dagster/schema/schema/charts/utils/utils.py

@@ -54,7 +54,7 @@ def create_definition_ref(definition: str, version: str = SupportedKubernetes.V1
 
 
 def create_json_schema_conditionals(
-    enum_type_to_config_name_mapping: Dict[Enum, str]
+    enum_type_to_config_name_mapping: Dict[Enum, str],
 ) -> List[dict]:
     return [
         {

pyproject.toml

@@ -231,7 +231,7 @@ select = [
 ]
 
 # Fail if Ruff is not running this version.
-required-version = "0.1.1"
+required-version = "0.1.7"
 
 [tool.ruff.flake8-builtins]
 

python_modules/dagster-graphql/dagster_graphql/implementation/fetch_asset_checks.py

@@ -25,7 +25,7 @@
 
 
 def asset_checks_iter(
-    context: WorkspaceRequestContext
+    context: WorkspaceRequestContext,
 ) -> Iterator[Tuple[CodeLocation, ExternalRepository, ExternalAssetCheck]]:
     for location, repository in repository_iter(context):
         for external_check in repository.external_repository_data.external_asset_checks or []:
@@ -59,7 +59,7 @@ def get_asset_check_execution_statuses_by_id(
         planned_execution_runs_by_run_id = {}
 
     def _status_for_execution(
-        execution: AssetCheckExecutionRecord
+        execution: AssetCheckExecutionRecord,
     ) -> AssetCheckExecutionResolvedStatus:
         record_status = execution.status
 

python_modules/dagster-graphql/dagster_graphql/implementation/utils.py

@@ -92,7 +92,7 @@ def _noop(_) -> None:
 class ErrorCapture:
     @staticmethod
     def default_on_exception(
-        exc_info: Tuple[Type[BaseException], BaseException, TracebackType]
+        exc_info: Tuple[Type[BaseException], BaseException, TracebackType],
     ) -> "GraphenePythonError":
         from dagster_graphql.schema.errors import GraphenePythonError
 
@@ -118,7 +118,7 @@ def watch(fn: Callable[[Exception], None]) -> Iterator[None]:
 
 
 def capture_error(
-    fn: Callable[P, T]
+    fn: Callable[P, T],
 ) -> Callable[P, Union[T, "GrapheneError", "GraphenePythonError"]]:
     def _fn(*args: P.args, **kwargs: P.kwargs) -> T:
         try:

python_modules/dagster/dagster/_core/container_context/config.py

@@ -21,7 +21,7 @@
 
 # Process fields shared by container contexts in all environments (docker / k8s / ecs etc.)
 def process_shared_container_context_config(
-    container_context_config: Optional[Mapping[str, Any]]
+    container_context_config: Optional[Mapping[str, Any]],
 ) -> Mapping[str, Any]:
     shared_container_context = process_config(
         SHARED_CONTAINER_CONTEXT_SCHEMA, container_context_config or {}