ensure not empty json matrix (#35)

Signed-off-by: vsoch <vsoch@users.noreply.github.com> Co-authored-by: vsoch <vsoch@users.noreply.github.com>
vsoch · Sep 10, 2021 · 375b96b · 375b96b
1 parent e90de88
commit 375b96b
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 5 deletions.
diff --git a/parsers/docker/dockerbuild.go → parsers/docker/build.go b/parsers/docker/dockerbuild.go → parsers/docker/build.go
@@ -208,11 +208,16 @@ func (s *DockerBuildParser) Parse(path string, changesOnly bool) error {
 
 		// Parse into json
 		outJson, _ := json.Marshal(results)
+		output := string(outJson)
 
-		// If we are running in a GitHub Action, set the outputs
+		// If it's empty, still provide an empty list
+		if output == "" {
+			output = "[]"
+		}
 
+		// If we are running in a GitHub Action, set the outputs
 		if utils.IsGitHubAction() {
-			fmt.Printf("::set-output name=dockerbuild_matrix::%s\n", string(outJson))
+			fmt.Printf("::set-output name=dockerbuild_matrix::%s\n", output)
 		}
 	}
 	return nil

diff --git a/parsers/docker/dockerfile.go b/parsers/docker/dockerfile.go
@@ -303,7 +303,11 @@ func (s *DockerfileParser) Parse(path string, dryrun bool, changesOnly bool) err
 	// If we are running in a GitHub Action, set the outputs
 	if utils.IsGitHubAction() {
 		outJson, _ := json.Marshal(results)
-		fmt.Printf("::set-output name=dockerfile_matrix::%s\n", string(outJson))
+		output := string(outJson)
+		if output == "" {
+			output = "[]"
+		}
+		fmt.Printf("::set-output name=dockerfile_matrix::%s\n", output)
 	}
 	return nil
 }
diff --git a/parsers/docker/dockerfilelist.go b/parsers/docker/dockerfilelist.go
@@ -70,7 +70,11 @@ func (s *DockerfileListParser) Parse(path string, includeArgs bool, changesOnly
 	// If we are running in a GitHub Action, set the outputs
 	if utils.IsGitHubAction() {
 		outJson, _ := json.Marshal(results)
-		fmt.Printf("::set-output name=dockerfilelist_matrix::%s\n", string(outJson))
+		output := string(outJson)
+		if output == "" {
+			output = "[]"
+		}
+		fmt.Printf("::set-output name=dockerfilelist_matrix::%s\n", output)
 	}
 	return nil
 }
diff --git a/parsers/docker/hierarchy.go b/parsers/docker/hierarchy.go
@@ -232,7 +232,11 @@ func (s *DockerHierarchyParser) Update(dryrun bool) error {
 	// If we are running in a GitHub Action, set the outputs
 	if utils.IsGitHubAction() {
 		outJson, _ := json.Marshal(results)
-		fmt.Printf("::set-output name=dockerhierarchy_matrix::%s\n", string(outJson))
+		output := string(outJson)
+		if output == "" {
+			output = "[]"
+		}
+		fmt.Printf("::set-output name=dockerhierarchy_matrix::%s\n", output)
 	}
 
 	return nil