-
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
[du] full sending on type annotations #21726
[du] full sending on type annotations #21726
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @tacastillo and the rest of your teammates on Graphite |
Deploy preview for dagster-university ready! ✅ Preview Built with commit 13aba55. |
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.
Left one comment but otherwise lgtm
docs/dagster-university/pages/dagster-essentials/lesson-3/defining-your-first-asset.md
Outdated
Show resolved
Hide resolved
Deploy preview for dagster-docs ready! Preview available at https://dagster-docs-ct1xf81vw-elementl.vercel.app Direct link to changed pages: |
Merge activity
|
73e96b1
to
bf879b5
Compare
b69af83
to
3d211ef
Compare
## Summary & Motivation ## How I Tested These Changes
Summary & Motivation
How I Tested These Changes