search

fonttools / fontbakery

🧁 A font quality assurance tool for everyone
https://fontbakery.readthedocs.io
Apache License 2.0
548 stars 99 forks source link

HTML and Markdown reporters are broken #4595

Closed guidoferreyra closed 6 months ago

guidoferreyra commented 6 months ago

The original title of this issue was: "HTML reporter is broken for FAIL results"

Observed behaviour

All the FAIL checks are being being ommited on the HTML report. See image.

imagen

Other checks that produces WARNS or ERROR are displayed correctly

glenda-tn commented 6 months ago

This is also broken for WARNS.

m4rc1e commented 6 months ago

Broken all round I'm afraid, https://github.com/google/fonts/pull/7384#issuecomment-1999294850

Since this has broken a few times now, it's probably worth writing a unit test.

felipesanches commented 6 months ago

Ouch! Sorry for that!

I'll try to fix it as quickly as possible

guidoferreyra commented 6 months ago

On my local clone of the repo I still don’t see the FAILS 🤔