Skip to content

Commit 9d76295

Browse files
authored
[dagster-dlt] fix typing error introduced in recent PR (#33113)
## Summary & Motivation Note: diverging from `AbstractSet` usage elsewhere due to warning: ``` python_modules/libraries/dagster-dlt/dagster_dlt/asset_decorator.py:2:1: UP035 `typing.AbstractSet` is deprecated, use `collections.abc.Set` instead | 1 | from collections.abc import Callable, Mapping, Sequence 2 | from typing import AbstractSet, Any, Optional | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ UP035 ``` ## How I Tested These Changes ## Changelog NOCHANGELOG
1 parent 6901b20 commit 9d76295

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

python_modules/libraries/dagster-dlt/dagster_dlt/asset_decorator.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
1-
from collections.abc import Callable, Mapping, Sequence
1+
from collections.abc import Callable, Mapping, Sequence, Set
22
from typing import Any, Optional
33

44
from dagster import (
5-
AbstractSet,
65
AssetsDefinition,
76
AssetSpec,
87
BackfillPolicy,
@@ -64,7 +63,7 @@ def dlt_assets(
6463
dagster_dlt_translator: Optional[DagsterDltTranslator] = None,
6564
partitions_def: Optional[PartitionsDefinition] = None,
6665
backfill_policy: Optional[BackfillPolicy] = None,
67-
hooks: Optional[AbstractSet[HookDefinition]] = None,
66+
hooks: Optional[Set[HookDefinition]] = None,
6867
op_tags: Optional[Mapping[str, Any]] = None,
6968
pool: Optional[str] = None,
7069
) -> Callable[[Callable[..., Any]], AssetsDefinition]:

0 commit comments

Comments
 (0)