Merge pull request #58 from aiyengar2/add_error_for_ambiguous_package_on_patch

aiyengar2 · web-flow · commit 579e62f8a7bc · 2021-08-24T10:34:38.000-07:00
Output error when package is not set for patch
diff --git a/main.go b/main.go
@@ -130,10 +130,11 @@ func prepareCharts(c *cli.Context) {
 
 func generatePatch(c *cli.Context) {
 	packages := getPackages()
-	for _, p := range packages {
-		if err := p.GeneratePatch(); err != nil {
-			logrus.Fatal(err)
-		}
+	if len(packages) != 1 {
+		logrus.Fatalf("No PACKAGE is set. Run export PACKAGE=<package> before running this command.")
+	}
+	if err := packages[0].GeneratePatch(); err != nil {
+		logrus.Fatal(err)
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -130,10 +130,11 @@ func prepareCharts(c *cli.Context) {`
`130`	`130`
`131`	`131`	`func generatePatch(c *cli.Context) {`
`132`	`132`	`packages := getPackages()`
`133`		`- for _, p := range packages {`
`134`		`- if err := p.GeneratePatch(); err != nil {`
`135`		`- logrus.Fatal(err)`
`136`		`- }`
	`133`	`+ if len(packages) != 1 {`
	`134`	`+ logrus.Fatalf("No PACKAGE is set. Run export PACKAGE=<package> before running this command.")`
	`135`	`+ }`
	`136`	`+ if err := packages[0].GeneratePatch(); err != nil {`
	`137`	`+ logrus.Fatal(err)`
`137`	`138`	`}`
`138`	`139`	`}`
`139`	`140`