Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Full revamp of Dockerfile #319

Closed
wants to merge 14 commits into from
Prev Previous commit
fix install_all dependencies
  • Loading branch information
balihb committed Jan 11, 2022
commit e7df8d0de7b978aea7284982f786c821c3725ec8
5 changes: 3 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -106,6 +106,7 @@ FROM python:${TAG}

ARG INFRACOST_VERSION=${INFRACOST_VERSION:-false}
ARG TERRAFORM_DOCS_VERSION=${TERRAFORM_DOCS_VERSION:-false}
ARG INSTALL_ALL=${INSTALL_ALL:-false}

ENV PYTHONUNBUFFERED=1

Expand All @@ -114,10 +115,10 @@ RUN apk add --no-cache \
git \
# All hooks deps
bash && \
if [ "${TERRAFORM_DOCS_VERSION}" = "false" ]; then \
if [ "${TERRAFORM_DOCS_VERSION}" != "false" ] || [ "${INSTALL_ALL}" != "false" ]; then \
apk add --no-cache perl \
; fi && \
if [ "${INFRACOST_VERSION}" = "false" ]; then \
if [ "${INFRACOST_VERSION}" != "false" ] || [ "${INSTALL_ALL}" != "false" ]; then \
apk add --no-cache jq \
; fi

Expand Down