saifxyzyz commented on PR #59998:
URL: https://github.com/apache/airflow/pull/59998#issuecomment-3707036535

   > > I found that the smtp user and password were deprecated to use smtp 
connections instead, so reverted changes in the config.yml; found no missing 
entries the config.yml file
   > 
   > Okay, reading the diff now makes more sense. But the changes are 
functionally equivalent to previous code, just in other structure. Or do I 
mis-read the diff? Result is now more like a refactoring result, not a change.
   
   yes, it is just refactoring, if the structural changes alone aren't worth 
the merge, I'm open to closing this, I'll leave the final call to you


-- 
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