Hide analysis hint when scan findings are present for non-Terminal renderers (#797)

egibs · web-flow · commit c5a86da69ade · 2025-02-12T09:15:45.000-06:00
Signed-off-by: egibs &lt;20933572+egibs@users.noreply.github.com&gt;
diff --git a/cmd/mal/mal.go b/cmd/mal/mal.go
@@ -565,7 +565,8 @@ func main() {
 						return err
 					}
 
-					if length > 0 && mc.Renderer.Name() != "Interactive" {
+					show := length > 0 && (mc.Renderer.Name() == "Simple" || strings.Contains(mc.Renderer.Name(), "Terminal"))
+					if show {
 						fmt.Fprintf(os.Stderr, "\n💡 For detailed analysis, try \"mal analyze <path>\"\n")
 					}
 

Original file line number	Diff line number	Diff line change
`@@ -565,7 +565,8 @@ func main() {`
`565`	`565`	`return err`
`566`	`566`	`}`
`567`	`567`
`568`		`- if length > 0 && mc.Renderer.Name() != "Interactive" {`
	`568`	`+ show := length > 0 && (mc.Renderer.Name() == "Simple" \|\| strings.Contains(mc.Renderer.Name(), "Terminal"))`
	`569`	`+ if show {`
`569`	`570`	`fmt.Fprintf(os.Stderr, "\n💡 For detailed analysis, try \"mal analyze <path>\"\n")`
`570`	`571`	`}`
`571`	`572`