[GitHub] [airflow] potiuk edited a comment on issue #12983: Add support for the new selective docs building in CI build-docs step

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

potiuk edited a comment on issue #12983:
URL: https://github.com/apache/airflow/issues/12983#issuecomment-742520793


   I do not think it is "we must do it now" :). However, doc building takes now 23 (!) minutes on CI, so especially in the case of the "no-tests needed" builds that do not run any of the unit tests, it is definitely the one that stands out.
   
   I think it has very little to do with #12282 to be honest because selective checks are NOT part of Breeze. They are fully separated and they have nothing to do with Breeze at all. The selective checks could have been rewritten in python as part of implementing the 'selective docs'. That might be actually a very good idea. 
   
   And I agree it is a bit dangerous, but so all the selective tests are (however we always have the master builds after merge that run all the tests and docs and this should remain after this change) 
   
   We can make the logic a bit more complex if we rewrite it in python to make some protection against it (for example check very quickly if there are no :doc: or other references to the provider package that has just been changed.
   
   Just saying - it's a great opportunity to both - get some real benefits (in terms of feedback time and CI job strain) and to do it in a 'future-proof' way (i.e. python).
   
   WDYT?
   


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

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