potiuk commented on a change in pull request #13409:
URL: https://github.com/apache/airflow/pull/13409#discussion_r550640644
##########
File path: Dockerfile
##########
@@ -245,6 +245,10 @@ ENV
INSTALL_PROVIDERS_FROM_SOURCES=${INSTALL_PROVIDERS_FROM_SOURCES}
ARG UPGRADE_TO_NEWER_DEPENDENCIES="false"
ENV UPGRADE_TO_NEWER_DEPENDENCIES=${UPGRADE_TO_NEWER_DEPENDENCIES}
+# Those are additional constraints that are needed for some extras but we do
not want to
Review comment:
We need to embed those and use pre-commit because we want to handle the
situation when somebody updates it in PR . IN this case the scripts from CI
will not be used (only master version) so we need to make sure that Dockerfile's
default value is the same as the ARG passed in this case.
----------------------------------------------------------------
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]
