(maint) Update output to include status info

Update the default output, as well as the Azure Pipelines output, to
include information about the status of the task, and all the skip
reason, if there is something.

This change brings consistency to the output after the addition of the
new GitHub Actions Report Printer.

Author: gep13

src/Cake.AzurePipelines.Module/AzurePipelinesReportPrinter.cs

@@ -1,8 +1,10 @@
-using System;
+using System;
 using System.IO;
+using System.Linq;
 using System.Text;
 using Cake.Common.Build;
 using Cake.Core;
+using Cake.Core.Diagnostics;
 using Cake.Core.IO;
 using Cake.Module.Shared;
 using JetBrains.Annotations;
@@ -53,6 +55,8 @@ public override void Write(CakeReport report)
 
         private void WriteToMarkdown(CakeReport report)
         {
+            var includeSkippedReasonColumn = report.Any(r => !string.IsNullOrEmpty(r.SkippedMessage));
+
             var maxTaskNameLength = 29;
             foreach (var item in report)
             {
@@ -67,13 +71,30 @@ private void WriteToMarkdown(CakeReport report)
 
             var sb = new StringBuilder();
             sb.AppendLine("");
-            sb.AppendLine("|Task|Duration|");
-            sb.AppendLine("|:---|-------:|");
+
+            if (includeSkippedReasonColumn)
+            {
+                sb.AppendLine("|Task|Duration|Status|Skip Reason|");
+                sb.AppendLine("|:---|-------:|:-----|:----------|");
+            }
+            else
+            {
+                sb.AppendLine("|Task|Duration|Status|");
+                sb.AppendLine("|:---|-------:|:-----|");
+            }
+
             foreach (var item in report)
             {
                 if (ShouldWriteTask(item))
                 {
-                    sb.AppendLine(string.Format(lineFormat, item.TaskName, FormatDuration(item)));
+                    if (includeSkippedReasonColumn)
+                    {
+                        sb.AppendLine(string.Format(lineFormat, item.TaskName, FormatDuration(item), item.ExecutionStatus.ToReportStatus(), item.SkippedMessage));
+                    }
+                    else
+                    {
+                        sb.AppendLine(string.Format(lineFormat, item.TaskName, FormatDuration(item), item.ExecutionStatus.ToReportStatus()));
+                    }
                 }
             }
 

src/Cake.Module.Shared/CakeReportPrinterBase.cs

@@ -63,6 +63,8 @@ public virtual void WriteSkippedStep(string name, Verbosity verbosity)
         /// <param name="report">The report to write.</param>
         protected void WriteToConsole(CakeReport report)
         {
+            var includeSkippedReasonColumn = report.Any(r => !string.IsNullOrEmpty(r.SkippedMessage));
+
             var maxTaskNameLength = 29;
             foreach (var item in report)
             {
@@ -78,7 +80,14 @@ protected void WriteToConsole(CakeReport report)
 
             // Write header.
             _console.WriteLine();
-            _console.WriteLine(lineFormat, "Task", "Duration");
+            if (includeSkippedReasonColumn)
+            {
+                _console.WriteLine(lineFormat, "Task", "Duration", "Status", "Skip Reason");
+            }
+            else
+            {
+                _console.WriteLine(lineFormat, "Task", "Duration", "Status");
+            }
             _console.WriteLine(new string('-', 20 + maxTaskNameLength));
 
             // Write task status.
@@ -87,7 +96,14 @@ protected void WriteToConsole(CakeReport report)
                 if (ShouldWriteTask(item))
                 {
                     _console.ForegroundColor = GetItemForegroundColor(item);
-                    _console.WriteLine(lineFormat, item.TaskName, FormatDuration(item));
+                    if (includeSkippedReasonColumn)
+                    {
+                        _console.WriteLine(lineFormat, item.TaskName, FormatDuration(item), item.ExecutionStatus.ToReportStatus(), item.SkippedMessage);
+                    }
+                    else
+                    {
+                        _console.WriteLine(lineFormat, item.TaskName, FormatDuration(item), item.ExecutionStatus.ToReportStatus());
+                    }
                 }
             }
 
@@ -142,7 +158,7 @@ protected static string FormatDuration(CakeReportEntry item)
         {
             if (item.ExecutionStatus == CakeTaskExecutionStatus.Skipped)
             {
-                return "Skipped";
+                return "-";
             }
 
             return FormatTime(item.Duration);