more github actions summary formatting

RageAgainstThePixel · Jan 29, 2023 · d45ae87 · d45ae87
1 parent 1ac8911
commit d45ae87
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 11 deletions.
diff --git a/Editor/Logging/GitHubActionsLogger.cs b/Editor/Logging/GitHubActionsLogger.cs
@@ -63,6 +63,9 @@ public override void GenerateBuildSummary(BuildReport buildReport, Stopwatch sto
 
             summaryWriter.WriteLine($"Total duration: {stopwatch.Elapsed:g}");
             summaryWriter.WriteLine($"Size: {FormatFileSize(buildReport.summary.totalSize)}");
+            summaryWriter.WriteLine("");
+            summaryWriter.WriteLine("## Logs");
+            summaryWriter.WriteLine("");
 
             switch (buildReport.summary.result)
             {
@@ -83,6 +86,10 @@ public override void GenerateBuildSummary(BuildReport buildReport, Stopwatch sto
             var totalBuildTime = TimeSpan.Zero;
             var stepNumber = 0;
 
+
+            summaryWriter.WriteLine("| log type | message |");
+            summaryWriter.WriteLine("| -------- | ------- |");
+
             foreach (var step in buildReport.steps)
             {
                 stepNumber++;
@@ -99,27 +106,22 @@ public override void GenerateBuildSummary(BuildReport buildReport, Stopwatch sto
                 Debug.Log(buildStepMessage);
 
                 var hasMessages = step.messages.Length > 0;
-                summaryWriter.WriteLine($"## {buildStepMessage}");
-                summaryWriter.WriteLine($"Duration: {step.duration:g}");
 
                 if (!hasMessages)
                 {
                     continue;
                 }
 
-                summaryWriter.WriteLine($"<details open><summary>{step.messages.Length} Log Messages</summary>");
-                summaryWriter.WriteLine("");
-                summaryWriter.WriteLine("| log type | message |");
-                summaryWriter.WriteLine("| -------- | ------- |");
-
                 foreach (var message in step.messages)
                 {
                     var logMessage = message.content.Replace("\n", string.Empty);
+                    logMessage = logMessage.Replace("\r", string.Empty);
                     logMessage = logMessage.Replace(Error, string.Empty);
                     logMessage = logMessage.Replace(Warning, string.Empty);
                     logMessage = logMessage.Replace(ErrorColor, string.Empty);
                     logMessage = logMessage.Replace(WarningColor, string.Empty);
                     logMessage = logMessage.Replace(ResetColor, string.Empty);
+                    logMessage = logMessage.Replace(LogColor, string.Empty);
 
                     summaryWriter.WriteLine($"| {message.type} | {logMessage} |");
 
@@ -140,9 +142,6 @@ public override void GenerateBuildSummary(BuildReport buildReport, Stopwatch sto
                             throw new ArgumentOutOfRangeException();
                     }
                 }
-
-                summaryWriter.WriteLine("</details>");
-                summaryWriter.WriteLine("");
             }
 
             summaryWriter.Close();

diff --git a/package.json b/package.json
@@ -3,7 +3,7 @@
   "displayName": "Build Pipeline Utilities",
   "description": "The Build Pipeline Utilities aims to give developers more tools and options when making builds with the command line or with continuous integration.",
   "keywords": [],
-  "version": "1.1.3",
+  "version": "1.1.4",
   "unity": "2019.4",
   "license": "MIT",
   "repository": {