cli: Use max() to determine latest version

This uses the max() builtin to determine the latest pipenv version
instead of sorting the available versions and taking the last version.

This is clearer, idiomatic, and avoids the need for intermediate lists.
This commit is contained in:
David Vo
2017-10-07 21:23:58 +11:00
parent a15fa58de0
commit 61290abbb3
+2 -2
View File
@@ -141,7 +141,7 @@ def check_for_updates():
"""Background thread -- beautiful, isn't it?"""
try:
r = requests.get('https://pypi.python.org/pypi/pipenv/json', timeout=0.5)
latest = sorted([semver.parse_version_info(v) for v in list(r.json()['releases'].keys())])[-1]
latest = max(map(semver.parse_version_info, r.json()['releases'].keys()))
current = semver.parse_version_info(__version__)
if latest > current:
@@ -161,7 +161,7 @@ def ensure_latest_self(user=False):
except requests.RequestException as e:
click.echo(crayons.red(e))
sys.exit(1)
latest = sorted([semver.parse_version_info(v) for v in list(r.json()['releases'].keys())])[-1]
latest = max(map(semver.parse_version_info, r.json()['releases'].keys()))
current = semver.parse_version_info(__version__)
if current < latest: