Skip to content

Commit

Permalink
Roll back deprecation of multi-asset sensor (#26390)
Browse files Browse the repository at this point in the history
Resolves #26223 

## Summary & Motivation

`@multi_asset_sensor` was deprecated with a message saying to use
`AutomationCondition`, but `AutomationCondition` cannot cover all of
`@multi_asset_sensor` uses cases (like running jobs), as discussed in
#26223.

## Changelog

Deprecation of `@multi_asset_sensor` has been rolled back.
  • Loading branch information
smackesey authored Dec 19, 2024
1 parent d9ddad4 commit 40807bf
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
from typing import TYPE_CHECKING, Any, Callable, Mapping, Optional, Sequence, Set, Union

import dagster._check as check
from dagster._annotations import deprecated
from dagster._core.definitions.asset_selection import AssetSelection, CoercibleToAssetSelection
from dagster._core.definitions.asset_sensor_definition import AssetSensorDefinition
from dagster._core.definitions.events import AssetKey
Expand Down Expand Up @@ -248,7 +247,6 @@ def _wrapped_fn(*args, **kwargs) -> Any:
return inner


@deprecated(breaking_version="2.0.0", additional_warn_text="use `AutomationConditions` instead")
def multi_asset_sensor(
monitored_assets: Union[Sequence[AssetKey], AssetSelection],
*,
Expand Down

0 comments on commit 40807bf

Please sign in to comment.