potiuk commented on PR #35862: URL: https://github.com/apache/airflow/pull/35862#issuecomment-1826790086
This is a logically-separated PR from #35830 as discussed in https://github.com/apache/airflow/pull/35830#pullrequestreview-1748817163 It's mostly revolving about making `breeze shell` command more usable as standalone "driver" to execute arbitrary commands. It's quite a bit smaller than #35830 and focuses only on making such execution possible - once this is complete, I think I will just rebase the #35830 and what will be left is actually switching to the new `breeze shell` command as execution driver by the few pre-commits we have already using breeze code in a `twisted` way. I still left a few minor "loosely-related" fixes. I could probably extract out a few of those in separate PRs if you think it makes sense ("Few other cleanups and minor fixes were also implemented as part of this change" above). But it would inflate number of PRs and since it is just `dev env`, It's not going to be cherry-picked individually, so I'd rather lave them in - there are one-liner fixes that are anyhow done close to change already being implemented. But if needed I can separate them. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
