Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[core-api][experimental] AutomationCondition.any_downstream_conditions #26707

Conversation

OwenKephart
Copy link
Contributor

Summary & Motivation

decision: experimental -> beta
reason: we might want to subtly change the default behavior of this (e.g. how far down the tree it looks)
docs exist: n/a

How I Tested These Changes

Changelog

Insert changelog entry or delete this section.

This was referenced Dec 23, 2024
@OwenKephart OwenKephart changed the title [core-apis][experimental] AutomationCondition.any_downstream_conditions [core-api][experimental] AutomationCondition.any_downstream_conditions Dec 23, 2024
@OwenKephart OwenKephart force-pushed the 12-23-_core-apis_experimental_observable_source_asset_multi_observable_source_asset branch from 72bc4c7 to 11433a6 Compare December 23, 2024 21:56
@OwenKephart OwenKephart force-pushed the 12-23-_core-apis_experimental_automationcondition.any_downstream_conditions branch from 24f2a17 to 676e8d0 Compare December 23, 2024 21:56
@OwenKephart OwenKephart mentioned this pull request Dec 31, 2024
@OwenKephart OwenKephart force-pushed the 12-23-_core-apis_experimental_observable_source_asset_multi_observable_source_asset branch from 11433a6 to 0ba9041 Compare December 31, 2024 18:48
@OwenKephart OwenKephart force-pushed the 12-23-_core-apis_experimental_automationcondition.any_downstream_conditions branch from 676e8d0 to 8b03dc6 Compare December 31, 2024 18:48
@OwenKephart OwenKephart marked this pull request as ready for review December 31, 2024 18:52
@OwenKephart OwenKephart force-pushed the 12-23-_core-apis_experimental_automationcondition.any_downstream_conditions branch from 8b03dc6 to ac72e5b Compare December 31, 2024 19:17
@OwenKephart OwenKephart changed the base branch from 12-23-_core-apis_experimental_observable_source_asset_multi_observable_source_asset to core-api-audit December 31, 2024 19:17
## Summary & Motivation

decision: experimental -> GA
reason: given that we've bought in heavily to MaterializeResult, it
seems natural to promote this pattern as well
docs exist: no (api only, needs to be added to:
https://docs.dagster.io/concepts/assets/asset-observations#asset-observations)

## How I Tested These Changes

## Changelog

> Insert changelog entry or delete this section.
@OwenKephart OwenKephart merged commit efa7c18 into core-api-audit Dec 31, 2024
1 check was pending
@OwenKephart OwenKephart deleted the 12-23-_core-apis_experimental_automationcondition.any_downstream_conditions branch December 31, 2024 19:26
OwenKephart added a commit that referenced this pull request Dec 31, 2024
#26707)

## Summary & Motivation

decision: experimental -> beta
reason: we might want to subtly change the default behavior of this
(e.g. how far down the tree it looks)
docs exist: n/a

## How I Tested These Changes

## Changelog

> Insert changelog entry or delete this section.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants