Skip to content

Commit

Permalink
Pin to pip 9.0.2 for pipenv users only
Browse files Browse the repository at this point in the history
This addresses an issue raised by @CaseyFeist during code review:

  Updating pip for pipenv users or requiring them to update without a
  heads up won't be a good experience (our version is old enough that
  they'll need to uninstall and reinstall pipenv locally to successfully
  update). If you can refactor this to stay pinned to current version for
  pipenv users only, I should be able to accept this (and the related
  project updates).

  heroku#833 (comment)
  • Loading branch information
cjolowicz committed Oct 3, 2019
1 parent 946f4d6 commit 346a67d
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions bin/compile
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,11 @@ for file in "$BUILD_DIR/runtime.txt" "$CACHE_DIR/.heroku/python-version" ; do
esac
done

if [[ -f "$BUILD_DIR/Pipfile" ]]; then
# Do not force pipenv users to re-install pipenv locally.
PIP_UPDATE="9.0.2"
fi

export DEFAULT_PYTHON_STACK PIP_UPDATE
export PY37 PY36 PY35 PY27 PY34

Expand Down

0 comments on commit 346a67d

Please sign in to comment.