checkpatch: only print output for issues

No need to print code blocks if there are no issues, save space for important info. Signed-off-by: Matthieu Baerts (NGI0) <[email protected]>
multipath-tcp · Feb 23, 2024 · b72e1d4 · b72e1d4
1 parent ebef5cc
commit b72e1d4
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/entrypoint.sh b/entrypoint.sh
@@ -646,10 +646,15 @@ _checkpatch() {
 		-g HEAD 2>&1 | tee "${TMPFILE}" >&2
 
 	{
-		echo "- Commit $(get_commit_md):"
-		echo "\`\`\`"
-		cat "${TMPFILE}"
-		echo "\`\`\`"
+		echo -n "- Commit $(get_commit_md):"
+		if ! grep -q "^total: 0 errors, 0 warnings, 0 checks" "${TMPFILE}"; then
+			echo
+			echo "\`\`\`"
+			cat "${TMPFILE}"
+			echo "\`\`\`"
+		else
+			echo " no checkpatch issue"
+		fi
 	 } >> "${CHECKPATCH_DETAILS}"
 
 	grep "^total:" "${TMPFILE}" | tail -n1