-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[dagster-airlift][tutorial] Assets forward tutorial #25193
Conversation
54700ea
to
ca29993
Compare
535c869
to
26141d9
Compare
ca29993
to
640d53a
Compare
26141d9
to
b4ccfe3
Compare
examples/experimental/dagster-airlift/examples/tutorial-example/README.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unless necessary for tests, I don't think we should have rebuild_customer_list_assets
as separate functions. Would rather have those inlined.
640d53a
to
7e46695
Compare
b4ccfe3
to
b069871
Compare
7e46695
to
bc2aaae
Compare
b069871
to
a944b9e
Compare
Yea, I don't think it's necessary anymore since we only have one top-level definitions object now. Before it would induce multiple to inline them. |
0a38773
to
9dabb6f
Compare
Summary & Motivation
Assets forward version of the tutorial. Also changes some of the wording
How I Tested These Changes
Existing tests
Changelog
NOCHANGELOG