[GitHub] [airflow] potiuk commented on a change in pull request #12693: Installing packages in our CI uses constraints

[GitBox <gi...@apache.org>]

potiuk commented on a change in pull request #12693:
URL: https://github.com/apache/airflow/pull/12693#discussion_r537058323



##########
File path: scripts/in_container/_in_container_utils.sh
##########
@@ -263,14 +263,16 @@ function install_airflow_from_wheel() {
     echo
     echo "Install airflow wheel package from dist"
     echo
-    pip install /dist/apache_airflow-*.whl >"${OUT_FILE_PRINTED_ON_ERROR}" 2>&1
+    pip install /dist/apache_airflow-*.whl --no-deps >"${OUT_FILE_PRINTED_ON_ERROR}" 2>&1

Review comment:
       Yep. I can add a comment, but this is easy to explain. By that time all the dependencies are already installed (we remove only the packages (airflow + airflow-providers) that were installed from sources, but their dependencies are already installed as needed as we installed 'devel_ci' extra. That's why we just need to install the packages, build and nothing else.




----------------------------------------------------------------
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:
users@infra.apache.org