use + rather than - for semver compliance

app/setup_handlers.go

@@ -17,13 +17,16 @@ import (
 // There may be multiple upgrade handlers configured on some releases if different
 // migrations needto be run in different environment
 func GetDefaultUpgradeHandlerVersion() string {
+	// semver must have v prefix, but we store without prefix
+	vVersion := "v" + constant.Version
+
 	// development builds always use the full version in the release handlers
-	if semver.Build(constant.Version) != "" || semver.Prerelease(constant.Version) != "" {
+	if semver.Build(vVersion) != "" || semver.Prerelease(vVersion) != "" {
 		return constant.Version
 	}
 
 	// release builds use just the major version (v22.0.0 -> v22)
-	return semver.Major(constant.Version)
+	return semver.Major(vVersion)
 }
 
 func SetupHandlers(app *App) {

cmd/zetacored/version.go

@@ -10,7 +10,7 @@ import (
 func UpgradeHandlerVersionCmd() *cobra.Command {
 	return &cobra.Command{
 		Use:   "upgrade-handler-version",
-		Short: "Print the upgrade handler version",
+		Short: "Print the default upgrade handler version",
 		Run: func(cmd *cobra.Command, args []string) {
 			fmt.Println(app.GetDefaultUpgradeHandlerVersion())
 		},

version.sh

@@ -22,8 +22,8 @@ short_commit=$(git rev-parse --short HEAD)
 
 # append -dirty for dirty builds
 if ! git diff --no-ext-diff --quiet --exit-code ; then
-    echo "0.0.${commit_timestamp}-${short_commit}-dirty"
+    echo "0.0.${commit_timestamp}+${short_commit}-dirty"
     exit
 fi
 
-echo "0.0.${commit_timestamp}-${short_commit}"
+echo "0.0.${commit_timestamp}+${short_commit}"