Skip to content

Commit ab89e49

Browse files
alexandre-paroissienCaseyFaist
authored andcommitted
Fix pipenv install twice in CI
1 parent c126f62 commit ab89e49

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

bin/steps/pipenv

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -60,10 +60,16 @@ if [ ! "$SKIP_PIPENV_INSTALL" ]; then
6060
# avoid this eager behavior.
6161
/app/.heroku/python/bin/pip install pipenv==$PIPENV_VERSION --upgrade --upgrade-strategy only-if-needed &> /dev/null
6262

63+
# Install the test dependencies, for CI.
64+
if [ "$INSTALL_TEST" ]; then
65+
puts-step "Installing test dependencies…"
66+
/app/.heroku/python/bin/pipenv install --dev --system --deploy 2>&1 | cleanup | indent
67+
6368
# Install the dependencies.
64-
if [[ ! -f Pipfile.lock ]]; then
69+
elif [[ ! -f Pipfile.lock ]]; then
6570
puts-step "Installing dependencies with Pipenv $PIPENV_VERSION"
6671
/app/.heroku/python/bin/pipenv install --system --skip-lock 2>&1 | indent
72+
6773
else
6874
pipenv-to-pip Pipfile.lock > requirements.txt
6975
"$BIN_DIR/steps/pip-uninstall"
@@ -73,12 +79,6 @@ if [ ! "$SKIP_PIPENV_INSTALL" ]; then
7379
puts-step "Installing dependencies with Pipenv $PIPENV_VERSION"
7480
/app/.heroku/python/bin/pipenv install --system --deploy 2>&1 | indent
7581
fi
76-
77-
# Install the test dependencies, for CI.
78-
if [ "$INSTALL_TEST" ]; then
79-
puts-step "Installing test dependencies…"
80-
/app/.heroku/python/bin/pipenv install --dev --system --deploy 2>&1 | cleanup | indent
81-
fi
8282
fi
8383
else
8484
export SKIP_PIP_INSTALL=1

0 commit comments

Comments
 (0)