-
Notifications
You must be signed in to change notification settings - Fork 124
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
Draft: Add config to TaskInstance
context
#700
base: main
Are you sure you want to change the base?
Conversation
👷 Deploy Preview for amazing-pothos-a3bca0 processing.
|
e672125
to
d47889a
Compare
d47889a
to
be5c79c
Compare
Getting type check failures because of pre python 3.9 available type hint syntax that is being used, like |
Thanks for working on this!
@wseaton it's possible to avoid this exception in previous versions of Python by using I'm changing this PR to draft mode - please, move it back to a non-draft PR once the branch is rebased and the tests are passing. |
hi @wseaton are you still working on this PR or could we close this? |
Description
forwards context from the parsed model config object to the Airflow
TaskInstance
Context
so it is available in callback functionsRelated Issue(s)
closes #698
Breaking Change?
not that I'm aware
Checklist
Would like guidance on where you'd like the tests for this, eg. mocked in the operators section or integration.