potiuk commented on a change in pull request #4936: [AIRFLOW-4115]
Multi-staging Aiflow Docker image [Step 1/3]
URL: https://github.com/apache/airflow/pull/4936#discussion_r268862273
##########
File path: setup.py
##########
@@ -252,7 +253,7 @@ def write_version(filename=os.path.join(*['airflow',
'qds-sdk>=1.9.6',
'rednose',
'requests_mock',
- 'flake8>=3.6.0',
+ 'virtualenv',
Review comment:
But on the other hand - what if someone actually wants to use the
PythonVirtualenvOperator ? Should not they actually have to be given a chance
to do pip install -e.[something,something,virtualenv] ? Maybe in this case
virtualenv deserves its own extra?
----------------------------------------------------------------
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