justinpakzad commented on PR #59912:
URL: https://github.com/apache/airflow/pull/59912#issuecomment-3697808705

   > LGTM, but I wonder if this is really intended. It looks like it should be. 
Also it would be great to get some tests covering it ?
   
   So there are already two tests in the 
`tests/task_sdk/bases/test_operator.py` file that cover the defaults and 
setting email_on_retry to False. I assumed that should be sufficient but happy 
to add more if needed.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to