potiuk commented on a change in pull request #7793: Verify that all providers 
packages are included
URL: https://github.com/apache/airflow/pull/7793#discussion_r396103978
 
 

 ##########
 File path: scripts/ci/_utils.sh
 ##########
 @@ -38,6 +38,9 @@ function verbose_docker {
 }
 
 function initialize_breeze_environment {
+    # default python version
+    PYTHON_VERSION=${PYTHON_VERSION:="3.6"}
 
 Review comment:
   Not really. The name here is misleading -  setup_default_breeze_variables is 
for breeze command. This one above initializes common environment that is used 
by both breeze and the CI scripts (CI scripts do no use Breeze as such - they 
share common methods with it). I will change the name to 
initialize_common_environment in a separate commit :)

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

Reply via email to