Merge pull request #1392 from glensc/log-details

Refactor: Put long node error inside html details
danger · Jul 31, 2023 · c561731 · c561731
2 parents b0e5f37 + 5855ac9
commit c561731
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/source/commands/utils/reporting.ts b/source/commands/utils/reporting.ts
@@ -1,7 +1,7 @@
 import { DangerResults } from "../../dsl/DangerResults"
 
 export const markdownCode = (string: string): string => `
-\`\`\`sh
+\`\`\`
 ${string}
 \`\`\`
 `

diff --git a/source/commands/utils/runDangerSubprocess.ts b/source/commands/utils/runDangerSubprocess.ts
@@ -127,7 +127,8 @@ export const runDangerSubprocess = (
       d(`Handling fail from subprocess`)
       process.exitCode = code
 
-      const failResults = resultsWithFailure(`\`${processDisplayName}\` failed.`, "### Log\n\n" + markdownCode(allLogs))
+      const moreMarkdown = "### Log\n\n<details>\n\n" + markdownCode(allLogs) + "\n\n</details>\n"
+      const failResults = resultsWithFailure(`\`${processDisplayName}\` failed.`, moreMarkdown)
       if (results) {
         results = mergeResults(results, failResults)
       } else {