potiuk commented on pull request #22713: URL: https://github.com/apache/airflow/pull/22713#issuecomment-1086918758
cc: @Bowrna @edithturn -> I have deeeply reviewed, tested and cleaned up the changes. This is really cool what we have, after I removed some duplications, cleaned up some naming, added some docstrings and reviewed and made sure all the parameters we have, we are 100% ready to replace old breeze and move the old breeze to legacy. This will be the next PR after the #22695 and this one is merged. My changes were mostly about beautifying things, refactoring/shuffling them around so that they make more sense and making sure we have no duplications. All the rest remained largely as You've implemented it. FANTASTIC JOB. Some cool screenshots follow. -- 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]
