Merge pull request #350 from vdemeester/version-command-no-warn

vdemeester · web-flow · commit c66e350ecd0e · 2016-08-10T15:03:12.000+02:00
Update Version signature to remove warning
diff --git a/cli/app/version.go b/cli/app/version.go
@@ -17,10 +17,10 @@ Built:        {{.BuildTime}}
 OS/Arch:      {{.Os}}/{{.Arch}}`
 
 // Version prints the libcompose version number and additionnal informations.
-func Version(c *cli.Context) {
+func Version(c *cli.Context) error {
 	if c.Bool("short") {
 		fmt.Println(version.VERSION)
-		return
+		return nil
 	}
 
 	tmpl, err := template.New("").Parse(versionTemplate)
@@ -48,5 +48,5 @@ func Version(c *cli.Context) {
 		logrus.Fatal(err)
 	}
 	fmt.Printf("\n")
-	return
+	return nil
 }

Original file line number	Diff line number	Diff line change
`@@ -17,10 +17,10 @@ Built: {{.BuildTime}}`
`17`	`17`	OS/Arch: {{.Os}}/{{.Arch}}`
`18`	`18`
`19`	`19`	`// Version prints the libcompose version number and additionnal informations.`
`20`		`-func Version(c *cli.Context) {`
	`20`	`+func Version(c *cli.Context) error {`
`21`	`21`	`if c.Bool("short") {`
`22`	`22`	`fmt.Println(version.VERSION)`
`23`		`- return`
	`23`	`+ return nil`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`tmpl, err := template.New("").Parse(versionTemplate)`
`@@ -48,5 +48,5 @@ func Version(c *cli.Context) {`
`48`	`48`	`logrus.Fatal(err)`
`49`	`49`	`}`
`50`	`50`	`fmt.Printf("\n")`
`51`		`- return`
	`51`	`+ return nil`
`52`	`52`	`}`