fix markdown/html reporters

(issue #4595)
fonttools · Mar 15, 2024 · 0aa38f3 · 0aa38f3
1 parent 558b430
commit 0aa38f3
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 12 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,7 @@ A more detailed list of changes is available in the corresponding milestones for
 
 ## Upcoming release: 0.12.0 (2024-Mar-??)
   - Documentation for profile and check writers has been rewritten.
+  - Fixed markdown/html reporters (issue #4595)
   - And also includes the changes from the previous release-candidates:
     - v0.12.0a1 (2024-Feb-14)
     - v0.12.0a2 (2024-Feb-21)

diff --git a/Lib/fontbakery/reporters/templates/markdown/checks.markdown b/Lib/fontbakery/reporters/templates/markdown/checks.markdown
@@ -1,8 +1,5 @@
-<details>
-    <summary>[{{checks |length}}] {{filename}}</summary>
-    <div>
-        {% for check in checks %}
-        {% include "check.markdown" %}
-        {% endfor %}
-    </div>
-</details>
+<details><summary>[{{checks |length}}] {{filename}}</summary>
+<div>{% for check in checks %}
+{% include "check.markdown" %}
+{% endfor %}</div>
+</details>
diff --git a/Lib/fontbakery/reporters/templates/markdown/main.markdown b/Lib/fontbakery/reporters/templates/markdown/main.markdown
@@ -35,10 +35,10 @@ These won't break the CI job for now, but will become effective after some time
 {% if total > 0 %}
 ### Summary
 
-| {%for level in summary.keys() %}{{level | emoticon }} {{level}} | {%endfor%} |
-|---|---|
-| {%for level in summary.keys() %}{{summary[level]}} | {%endfor%} |
-| {%for level in summary.keys() %}{{summary[level] | percent_of(total )}} | {%endfor%} |
+| {%for level in summary.keys() %}{{level | emoticon }} {{level}} | {%endfor%}
+| {%for level in summary.keys() %}---|{%endfor%}
+| {%for level in summary.keys() %}{{summary[level]}} | {%endfor%}
+| {%for level in summary.keys() %}{{summary[level] | percent_of(total )}} | {%endfor%}
 {% endif %}
 
 {% if omitted %}