-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
cmd/use cmd/install: add --latest to directly install latest release tag #86
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In addition to these minor comments, this doesn't work when sortAscending = true.
The easiest way to handle this is to add a |
Co-authored-by: Joe Mooring <[email protected]>
Co-authored-by: Joe Mooring <[email protected]>
Co-authored-by: Joe Mooring <[email protected]>
I guess I wasn't clear. Store the latest version (a string) in the struct. |
you are always too fast for me - thought we can skip processing the all the tasks - skip futher explanation. will update , sry |
and you are definitely right: much cleaner implementation - sry for the inconvenience |
Co-authored-by: Joe Mooring <[email protected]>
Co-authored-by: Joe Mooring <[email protected]>
OK, this looks good. Thanks. |
implements: #85
--latest
switchif latest
reads bad for me because it does not check if its latest but tells to use the latest version