[GitHub] [airflow] potiuk commented on pull request #12527: Move providers references to separate package + Spell-check in a common job with docs-build

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

potiuk commented on pull request #12527:
URL: https://github.com/apache/airflow/pull/12527#issuecomment-731564388


   > The documentation for building the documentation has been. moved to [/docs/README.rst](https://github.com/apache/airflow/blob/master/docs/README.rst) to make it easier to find. For now this file may not be very visible, but I will clean up this directory a bit soon and it will be the only .rst file in this directory. All other files will be in subdirectories following to the structure of the output directory.
   
   So why do we neet do join it in one job then? What are the benefits?
   
   If they are split into two separate jobs, they run in parallel and you can get the feedback much faster if you run a small doc-only PR and there are not many PRs in the queue.
   
   In case we run those two jobs in parallel, rather than waiting 2:40 seconds user will have to wait 1:32. I thin this is a good reason enough to keep those jobs as two separate jobs. Unless I am missing something.
   
   
   
   


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