Merge pull request #16 from vibrato/version-fix

Changed to follow Cobra convention for version flag on cli
servian · Aug 13, 2018 · bb07f81 · bb07f81
2 parents f817f1d + c675903
commit bb07f81
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 43 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -31,7 +31,7 @@ jobs:
             mkdir output
             go build -v -o TechTestApp .
             ls
-            ./TechTestApp version | sed -e 's/Version: /v./' > versionfile
+            ./TechTestApp --version | sed -e 's/TechTestApp version /v./' > versionfile
 
 
       - run: 

diff --git a/cmd/root.go b/cmd/root.go
@@ -54,6 +54,7 @@ var rootCmd = &cobra.Command{
 This application is used as part of testing potential candiates at Vibrato.
 
 Please visit http://vibrato.com.au for more details`,
+	Version: "0.3.1",
 }
 
 // Execute adds all child commands to the root command and sets flags appropriately.

diff --git a/cmd/version.go b/cmd/version.go
diff --git a/doc/readme.md b/doc/readme.md
@@ -120,7 +120,7 @@ Continuous integration is managed through circleci and the build on the master b
 
 Releases are deployed and managed through github, it's an automated process that is executed through the CI solution
 
-To create a new release, update `../cmd/version.go` with the new version and merge that into the master branch.
+To create a new release, update `../cmd/root.go` with the new version and merge that into the master branch.
 
 The commit message on the merge, will be the release message, so make sure it contains the release notes.