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

Check the versions in use against the latest published versions #9

Merged
merged 3 commits into from
Aug 30, 2023

Conversation

joseph-palmer
Copy link
Contributor

Summary

This PR adds checks that compare the CLI and API versions in use to the latest release and display a message if the versions in use are not the latest version.

The API version is checked against the tag of the latest docker image for the API (ignoring .latest and .prerelease to just look at the numbers, which should match the pyproject.toml version number).

The CLI is checked against the release tags of the CLI in this repo.

Issues

See alan-turing-institute/RCTab/issues/15

How to run

rctab --version will display You are using an old version of the RCTab CLI. The latest version is {latest.tag}. Pull the latest version from GitHub (https:://github.com/alan-turing-institute/rctab-cli) if the CLI version does not match the tag of the latest release. I will also display You are using an old version of the RCTab API. The latest version is {latest.version}. Restart the webapp in the Azure portal to update to the latest version. if the API version does not match the tag of the latest image on Docker Hub.

@joseph-palmer joseph-palmer requested a review from Iain-S August 25, 2023 13:42
Copy link
Contributor

@pwochner pwochner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a comment in README after installation to run rctab --version

@pwochner pwochner merged commit 85d1064 into main Aug 30, 2023
@pwochner pwochner deleted the check-for-version-updates branch August 30, 2023 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants