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

asset checks stepKey gql #17055

Merged
merged 2 commits into from
Oct 9, 2023
Merged

Conversation

johannkm
Copy link
Contributor

@johannkm johannkm commented Oct 6, 2023

Surface the asset check step key, so we can link to the step in runs

@johannkm
Copy link
Contributor Author

johannkm commented Oct 6, 2023

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@johannkm johannkm mentioned this pull request Oct 6, 2023
@johannkm johannkm force-pushed the johann/10-05-asset_checks_stepKey_gql branch from 4fdfe28 to 60798bc Compare October 6, 2023 04:19
@github-actions
Copy link

github-actions bot commented Oct 6, 2023

Deploy preview for dagit-core-storybook ready!

✅ Preview
https://dagit-core-storybook-nsui55uqm-elementl.vercel.app
https://johann-10-05-asset-checks-stepKey-gql.core-storybook.dagster-docs.io

Built with commit 31f8b05.
This pull request is being automatically deployed with vercel-action

@johannkm johannkm force-pushed the johann/09-23-Store_asset_check_planned_events_in_AssetCheckEvaluationsTable branch from e1b6d52 to 458d7a4 Compare October 6, 2023 04:57
@johannkm johannkm force-pushed the johann/10-05-asset_checks_stepKey_gql branch from 60798bc to 31f8b05 Compare October 6, 2023 04:57
Base automatically changed from johann/09-23-Store_asset_check_planned_events_in_AssetCheckEvaluationsTable to master October 9, 2023 13:28
@johannkm johannkm merged commit 4b1ae14 into master Oct 9, 2023
1 check passed
@johannkm johannkm deleted the johann/10-05-asset_checks_stepKey_gql branch October 9, 2023 13:28
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