Merge pull request #218 from danieljamesrees/fix_github_version_param

Darren Jones · web-flow · commit 1c9334f78d7b · 2018-05-14T14:24:36.000+01:00
Use version number not file as parameter
diff --git a/ci/tasks/publish-draft-release/publish-draft-release.go b/ci/tasks/publish-draft-release/publish-draft-release.go
@@ -219,5 +219,6 @@ func main() {
 
 	if err != nil {
 		os.Stderr.WriteString(fmt.Sprintf("Failed to publish draft release - %s\n", err))
+		os.Exit(1)
 	}
 }
diff --git a/ci/tasks/publish-draft-release/publish-draft-release.yml b/ci/tasks/publish-draft-release/publish-draft-release.yml
@@ -12,4 +12,4 @@ run:
   - | 
     cp -R devtools-boshrelease/ci/tasks/publish-draft-release /go/src/
     cd /go/src/publish-draft-release
-    go run publish-draft-release.go ${OWNER} ${REPO} ${VERSION} ${BRANCH}
+    go run publish-draft-release.go "${OWNER}" "${REPO}" "$(<${VERSION})" "${BRANCH}"

Original file line number	Diff line number	Diff line change
`@@ -219,5 +219,6 @@ func main() {`
`219`	`219`
`220`	`220`	`if err != nil {`
`221`	`221`	`os.Stderr.WriteString(fmt.Sprintf("Failed to publish draft release - %s\n", err))`
	`222`	`+ os.Exit(1)`
`222`	`223`	`}`
`223`	`224`	`}`