Skip to content

Commit

Permalink
fix(version): Version check now removes "v" first character from tag (#…
Browse files Browse the repository at this point in the history
…129)

This avoid version check to always warn a new version is available

Close #129
  • Loading branch information
Toilal committed Dec 16, 2020
1 parent 7d64531 commit 71a3567
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 6 deletions.
6 changes: 3 additions & 3 deletions ddb/__main__.py
Expand Up @@ -32,7 +32,7 @@
from ddb.phase import phases
from ddb.registry import Registry, RegistryObject
from ddb.service import services
from ddb.utils.release import ddb_repository, get_lastest_release_version
from ddb.utils.release import ddb_repository, get_latest_release_version
from ddb.utils.table_display import get_table_display
from ddb.feature.core import ConfigureSecondPassException

Expand Down Expand Up @@ -374,7 +374,7 @@ def main(args: Optional[Sequence[str]] = None,
]
]

last_release = get_lastest_release_version()
last_release = get_latest_release_version()
if last_release and __version__ < last_release:
version_content.append([
'',
Expand Down Expand Up @@ -440,7 +440,7 @@ def _check_for_update():
today = date.today()

if last_check is None or last_check < today:
last_release = get_lastest_release_version()
last_release = get_latest_release_version()
if last_release and __version__ < last_release:
header = 'A new version is available : {}'.format(last_release)
content = [[
Expand Down
9 changes: 6 additions & 3 deletions ddb/utils/release.py
Expand Up @@ -3,14 +3,17 @@
ddb_repository = 'gfi-centre-ouest/docker-devbox-ddb'


def get_lastest_release_version():
def get_latest_release_version():
"""
Retrieve latest release version from GitHub API
:return: the json flux from GitHub API
:return: Version from tag_name retrieved from GitHub API
"""
response = requests.get('https://api.github.com/repos/{}/releases/latest'.format(ddb_repository))
try:
response.raise_for_status()
return response.json().get('tag_name')
tag_name = response.json().get('tag_name')
if tag_name and tag_name.startswith('v'):
tag_name = tag_name[1:]
return tag_name
except: # pylint:disable=bare-except
return None

0 comments on commit 71a3567

Please sign in to comment.