dimberman commented on a change in pull request #6266: [AIRFLOW-2439]
Production Docker image support including refactoring of build scripts
URL: https://github.com/apache/airflow/pull/6266#discussion_r332504780
##########
File path: scripts/ci/ci_refresh_pylint_todo.sh
##########
@@ -23,13 +23,21 @@ MY_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
export AIRFLOW_CI_SILENT=${AIRFLOW_CI_SILENT:="false"}
export FORCE_ANSWER_TO_QUESTIONS=quit
-# shellcheck source=scripts/ci/_utils.sh
-. "${MY_DIR}/_utils.sh"
-
-basic_sanity_checks
+# shellcheck source=scripts/ci/utils/_init.sh
+. "${MY_DIR}/utils/_init.sh"
+# shellcheck source=scripts/ci/utils/_build.sh
+. "${MY_DIR}/utils/_build.sh"
+# shellcheck source=scripts/ci/utils/_run.sh
+. "${MY_DIR}/utils/_run.sh"
Review comment:
These lines seem to be repeated a lot. Is there any way of merging this into
a single command and sourcing it?
----------------------------------------------------------------
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
