Skip to content

Commit

Permalink
Fix script path and remove unused variable
Browse files Browse the repository at this point in the history
  • Loading branch information
bugraoz93 committed Nov 16, 2024
1 parent d90ef38 commit 9df949a
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 5 deletions.
3 changes: 1 addition & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -1403,7 +1403,6 @@ COPY <<"EOF" /install_yarn_dependencies_from_branch_tip.sh

: "${AIRFLOW_REPO:?Should be set}"
: "${AIRFLOW_BRANCH:?Should be set}"
: "${AIRFLOW_SOURCES:?Should be set}"

function install_yarn_dependencies_from_branch_tip() {
echo
Expand Down Expand Up @@ -1621,7 +1620,7 @@ RUN bash /scripts/docker/install_packaging_tools.sh; \

# We are installing Yarn dependencies here to make sure they are cached in the layer
RUN if [[ ${AIRFLOW_PRE_CACHED_YARN_PACKAGES} == "true" ]]; then \
bash /install_yarn_dependencies_from_branch_tip.sh; \
bash /scripts/docker/install_yarn_dependencies_from_branch_tip.sh; \
fi


Expand Down
3 changes: 1 addition & 2 deletions Dockerfile.ci
Original file line number Diff line number Diff line change
Expand Up @@ -1178,7 +1178,6 @@ COPY <<"EOF" /install_yarn_dependencies_from_branch_tip.sh

: "${AIRFLOW_REPO:?Should be set}"
: "${AIRFLOW_BRANCH:?Should be set}"
: "${AIRFLOW_SOURCES:?Should be set}"

function install_yarn_dependencies_from_branch_tip() {
echo
Expand Down Expand Up @@ -1390,7 +1389,7 @@ RUN bash /scripts/docker/install_packaging_tools.sh; \

# We are installing Yarn dependencies here to make sure they are cached in the layer
RUN if [[ ${AIRFLOW_PRE_CACHED_YARN_PACKAGES} == "true" ]]; then \
bash /install_yarn_dependencies_from_branch_tip.sh; \
bash /scripts/docker/install_yarn_dependencies_from_branch_tip.sh; \
fi

# Here we fix the versions so all subsequent commands will use the versions
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@

: "${AIRFLOW_REPO:?Should be set}"
: "${AIRFLOW_BRANCH:?Should be set}"
: "${AIRFLOW_SOURCES:?Should be set}"

function install_yarn_dependencies_from_branch_tip() {
echo
Expand Down

0 comments on commit 9df949a

Please sign in to comment.