zhongjiajie commented on issue #4897: [AIRFLOW-4062] Clear install extra package command URL: https://github.com/apache/airflow/pull/4897#issuecomment-471554625 @XD-DENG Think about it for a while, I want to discuss PR once again. I have two points. * We should compatible our command in many operating system and many type of shell, and this change will not break others and could use in `zsh` * We have some other `pip install` using `''` between extra package, just like below. https://github.com/apache/airflow/blame/master/docs/howto/executor/use-celery.rst#L64 https://github.com/apache/airflow/blame/master/CONTRIBUTING.md#L132
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
