Kiretori commented on PR #54680: URL: https://github.com/apache/airflow/pull/54680#issuecomment-3214539632
> > > Would it be possible to add a unit test ? > > > > > > I don't think that's necessary for this change. I just added some guard rails with an existence check, if there are already some unit tests for the `SmtpNotifier` they should still pass. If a unit test is mandatory for this PR to get merged I could look into that. > > Na.. It's ok . Templating is not as **crucial** to be tested. Okay, I guess it's ready to be merged now. -- 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]
