Address some feedback

Signed-off-by: Cintia Sanchez Garcia <[email protected]>

dco2/templates/how_to_fix.md

@@ -0,0 +1,7 @@
+{%~ if commits|contains_error([CommitError::SignOffNotFound, CommitError::SignOffMismatch]) +%}
+  ## How to fix missing or invalid sign-offs
+
+  {%~ include "remediation.md" +%}
+
+  {%~ include "signoff.md" +%}
+{% endif %}

dco2/templates/output.md

@@ -18,15 +18,9 @@
 
   {%~ include "summary.md" +%}
 
-  {%~ include "details.md" +%}
+  {%~ include "errors_details.md" +%}
 
-  {%~ if commits|contains_error([CommitError::SignOffNotFound, CommitError::SignOffMismatch]) +%}
-    {{~ "## How to fix missing or invalid sign-offs" +}}
-
-    {%~ include "remediation.md" +%}
-
-    {%~ include "signoff.md" +%}
-  {% endif %}
+  {%~ include "how_to_fix.md" +%}
 
 {% endif %}
 

dco2/templates/summary.md

@@ -7,9 +7,10 @@
 {%+ for entry in commits %}
   {% let (sha_truncated, rest) = entry.commit.sha.split_at(8) %}
   {% let message = entry.commit.message.split_once('\n') %}
+  {% let subject = entry.commit.message.lines().next().unwrap_or_default() %}
   {% if entry.errors.is_empty() %}
-    | :green_circle: | [{{ sha_truncated }}]({{ entry.commit.html_url }}) | {% if let Some((subject, rest)) = message %}{{ subject|truncate(50) }}{% else %}{{ entry.commit.message|truncate(50) }}{% endif %} | {% if let Some(success_reason) = entry.success_reason %}`{{ success_reason|capitalize }}`{% endif %} |
+    | :green_circle: | [{{ sha_truncated }}]({{ entry.commit.html_url }}) | {{ subject|truncate(50) }} | {% if let Some(success_reason) = entry.success_reason %}`{{ success_reason|capitalize }}`{% endif %} |
   {% else %}
-    | :red_circle: | [{{ sha_truncated }}]({{ entry.commit.html_url }}) | {% if let Some((subject, rest)) = message %}{{ subject|truncate(50) }}{% else %}{{ entry.commit.message|truncate(50) }}{% endif %} |{% for error in entry.errors %}{% if !loop.first %}{{ " " +}}{% endif %}`{{ error|capitalize }}`{% endfor %} |
+    | :red_circle: | [{{ sha_truncated }}]({{ entry.commit.html_url }}) | {{ subject|truncate(50) }} |{% for error in entry.errors %}{% if !loop.first %}{{ " " +}}{% endif %}`{{ error|capitalize }}`{% endfor %} |
   {% endif +%}
 {%+ endfor +%}