-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
[AIRFLOW-5640] fix get_email_address_list types #6315
[AIRFLOW-5640] fix get_email_address_list types #6315
Conversation
2052a38
to
ac87ff9
Compare
ac87ff9
to
23bc2c5
Compare
Codecov Report
@@ Coverage Diff @@
## master #6315 +/- ##
==========================================
- Coverage 80.09% 80.09% -0.01%
==========================================
Files 616 616
Lines 35795 35803 +8
==========================================
+ Hits 28670 28676 +6
- Misses 7125 7127 +2
Continue to review full report at Codecov.
|
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.
Nice!
Btw, I don't understand the code coverage change report. The files where coverage drop is reported are not touched by this PR. The overall coverage shouldn't be impacted by this PR at all. |
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.
See comments
23bc2c5
to
6e94810
Compare
Thanks for your comments @ashb! I addressed all of them - please have another look :) |
390c7d9
to
d025897
Compare
d025897
to
7ee1d23
Compare
🤦♂ Yes of course. Ready for another round @ashb - now with succeeding tests |
944ef40
to
1e7706b
Compare
For the past 2 days I've been restarting the CI build over and over again - some tests are timing out nonetheless... From checking other PRs, this doesn't seem to be related to my code change. @ashb Can you please have another look at my changes? Thanks! |
Hey @SaturnFromTitan - please rebase to latest master - we've just merged a workaround for failing Kubernetes tests which should make your test pass (while working on a permanent fix) |
* Refactored get_email_address_list to have a better separation between string handling and other iterables. * Explicitely casting get_email_address_list argument to a list in case the argument was an iterable. This enables direct support for tuples, sets or the like. * Fixed type annotation of email parameter of BaseOperator to show that iterables are directly supported. * Added docstring entries for email, email_on_retry, email_on_failure and queue in BaseOperator.
1e7706b
to
c12e7f9
Compare
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.
Nice.
…6315) * Refactored get_email_address_list to have a better separation between string handling and other iterables. * Explicitely casting get_email_address_list argument to a list in case the argument was an iterable. This enables direct support for tuples, sets or the like. * Fixed type annotation of email parameter of BaseOperator to show that iterables are directly supported. * Added docstring entries for email, email_on_retry, email_on_failure and queue in BaseOperator. (cherry picked from commit 9ec562f)
…6315) * Refactored get_email_address_list to have a better separation between string handling and other iterables. * Explicitely casting get_email_address_list argument to a list in case the argument was an iterable. This enables direct support for tuples, sets or the like. * Fixed type annotation of email parameter of BaseOperator to show that iterables are directly supported. * Added docstring entries for email, email_on_retry, email_on_failure and queue in BaseOperator. (cherry picked from commit 9ec562f)
…6315) * Refactored get_email_address_list to have a better separation between string handling and other iterables. * Explicitely casting get_email_address_list argument to a list in case the argument was an iterable. This enables direct support for tuples, sets or the like. * Fixed type annotation of email parameter of BaseOperator to show that iterables are directly supported. * Added docstring entries for email, email_on_retry, email_on_failure and queue in BaseOperator. (cherry picked from commit 9ec562f)
…6315) * Refactored get_email_address_list to have a better separation between string handling and other iterables. * Explicitely casting get_email_address_list argument to a list in case the argument was an iterable. This enables direct support for tuples, sets or the like. * Fixed type annotation of email parameter of BaseOperator to show that iterables are directly supported. * Added docstring entries for email, email_on_retry, email_on_failure and queue in BaseOperator. (cherry picked from commit 9ec562f)
Make sure you have checked all steps below.
Jira
Tests
Commits