Skip to content

Conversation

nol166
Copy link
Contributor

@nol166 nol166 commented Mar 27, 2019

Describe in detail the problem you had and how this PR fixes it

Checks to see if there is a newer release available when running the CLI

Is there an open issue you can link to?

Fixes #213.

Copy link
Contributor

@NGTmeaty NGTmeaty left a comment

Choose a reason for hiding this comment

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

Other than this LGTM!

@NGTmeaty
Copy link
Contributor

The Docker version we compile doesn't get a version number. Might be a different issue, but it'd be nice to get version checking in there too :)

@nol166
Copy link
Contributor Author

nol166 commented Mar 28, 2019

The Docker version we compile doesn't get a version number. Might be a different issue, but it'd be nice to get version checking in there too :)

The docker image eventually inherits the source code from the binary so as soon as the CI catches up (if and when) this is merged it should work for the docker image as well.

@nol166 nol166 removed the wip label Mar 28, 2019
@nol166 nol166 requested a review from kylecarbs March 28, 2019 20:41
@nol166 nol166 added the enhancement Some improvement that isn't a feature label Apr 1, 2019
@nol166
Copy link
Contributor Author

nol166 commented Apr 17, 2019

resolved conflicts

@kylecarbs kylecarbs closed this Jun 19, 2019
@kylecarbs kylecarbs deleted the check-version branch June 19, 2019 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Some improvement that isn't a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add release number in CLI, tell you when to update.
4 participants