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/08/08 07:32:35 UTC

[GitHub] [airflow] potiuk commented on pull request #25410: Consolidate to one `schedule` param

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

   > Wow, 356 files changed. Would it be a good idea to split out the part that consolidates arguement handling, and change the examples and documentation in a later PR?
   
   I'd prefer to keep those together, even if it's a big change. There is a little value in splitting them, Especially with the new Github Review experience with "change file tree" it's easy to separate out the "code" from "documentation changes during review - and keeping docs and code changes in the same PR makes it better easier to cherry-pick and reason about.


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