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
dev-doctor: Bump minimum hub version requirement for backporting #16734
Conversation
@@ -187,7 +187,7 @@ func rootCmdRun(cmd *cobra.Command, args []string) { | |||
ifNotFound: checkError, | |||
versionArgs: []string{"--version"}, | |||
versionRegexp: regexp.MustCompile(`hub\s+version\s+(\d+.\d+\.\d+)`), | |||
minVersion: &semver.Version{Major: 2, Minor: 0, Patch: 0}, | |||
minVersion: &semver.Version{Major: 2, Minor: 14, Patch: 0}, |
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.
While at it, shall we also suggest steps to install this particular version?
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.
Good idea - added hint.
Backporting scripts need a version of github.com/github/hub that support the api command. The version of hub distributed with Ubuntu 20.04 LTS is 2.7.0, which is too old. Bump the minimum hub version to 2.14 as this is definitely new enough and was released 18 months ago. Signed-off-by: Tom Payne <tom@isovalent.com>
56c951b
to
67846ac
Compare
@@ -187,7 +187,8 @@ func rootCmdRun(cmd *cobra.Command, args []string) { | |||
ifNotFound: checkError, | |||
versionArgs: []string{"--version"}, | |||
versionRegexp: regexp.MustCompile(`hub\s+version\s+(\d+.\d+\.\d+)`), | |||
minVersion: &semver.Version{Major: 2, Minor: 0, Patch: 0}, | |||
minVersion: &semver.Version{Major: 2, Minor: 14, Patch: 0}, | |||
hint: `Download the latest version from https://github.com/github/hub/releases.`, |
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.
This doesn't seem to have installation steps. Perhaps this link - https://github.com/github/hub? I'll leave it to you.
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.
The https://github.com/github/hub link actually contains incorrect installation steps for us. It recommends installing hub
with your distro's package manager, which will generally get you a version of hub
that's too old.
Backporting scripts need a version of github.com/github/hub that support
the api command. The version of hub distributed with Ubuntu 20.04 LTS is
2.7.0, which is too old. Bump the minimum hub version to 2.14 as this is
definitely new enough and was released 18 months ago.
Signed-off-by: Tom Payne tom@isovalent.com