You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "o-nikolas (via GitHub)" <gi...@apache.org> on 2023/03/07 23:53:03 UTC

[GitHub] [airflow] o-nikolas commented on pull request #29055: [AIP-51] Executors vending CLI commands

o-nikolas commented on PR #29055:
URL: https://github.com/apache/airflow/pull/29055#issuecomment-1459047776

   > BTW. It's ok to merge it as a single squashed PR I think
   
   Hey @potiuk 
   
   I had already started pulling out the commit which refactors `cli_parser.py` into it's own PR which can be reviewed here #29962
   
   
   We can review and approve that PR and then once it's merged, I'll rebase this PR (which will just represent the Executor related changes). I think that will be cleaner from a git log perspective. Since the two sets of changes are related, but mostly mutually exclusive. 
   
   Thoughts?


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