-
Notifications
You must be signed in to change notification settings - Fork 179
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
Add Python 3+ type-annotations #4
Conversation
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.
@y2k-shubham Thanks for the PR. This looks great! Are you using dag-factory
at Zomato?
@ajbosco thanks a ton! (that was quick) |
@y2k-shubham Which one did you choose at Zomato - dag-factory or boundary-layer? |
@Jaikant we didnt picky
|
Awesome. Sounds like you did all the right things. I am still in two minds about adopting airflow and somewhere I feel dagster-io has better abstractions built in. |
Use
typing
module to add Type-Annotations as per Python 3+