[GitHub] [airflow] potiuk commented on pull request #22327: Rewrite Selective Check in Python

[GitBox <gi...@apache.org>]

potiuk commented on PR #22327:
URL: https://github.com/apache/airflow/pull/22327#issuecomment-1108208623

   Hey @edithturn - After #23193 and #23205  - this should be part of https://github.com/apache/airflow/blob/main/dev/breeze/src/airflow_breeze/commands/configuration_and_maintenance.py
   
   Eventually it turned out that instead of standalone "freespace" etc. incorporating those tools into breeze command seems to be a much better and consistent solution.
   
   I think maybe simply let's start "small" -> let's merge a tool first that alllows to print "changed files" list as first step?
   
   for example `breeze selective-checks list-changed-files` :) ? . Click has the options to use sub-commands we have not used it so far) but this one is an obvious candidate). We will have to do the same for "kind" commands - so this might be a good opportunity for you to see how it is done.
   
   Then I could attempt to split this change into a number of smallers "selective check tasks" to make it easier and possibly parallelise the work. Same with kind :).


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