You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/04/03 18:01:11 UTC

[GitHub] [airflow] potiuk commented on pull request #22713: Prepare Breeze2 for prime time :)

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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org