fonttools / fontbakery

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

[com.google.fonts/check/description/has_article] should not require an empty Description file #4702

Closed vv-monsalve closed 1 month ago

vv-monsalve commented 1 month ago

Observed behavior

🔥 FAIL Check for presence of an ARTICLE.en_us.html file 🔥 FAIL This font has both a DESCRIPTION.en_us.html file and an ARTICLE.en_us.html file. In this case the description must be empty.

The above Fail was reported on this CI run. However, in a private chat @@nathan-williams said the empty DESCRIPTION.en_us.html file was no longer required when an ARTICLE.en_us.html is provided; "we can have just the ARTICLE file(s) where appropriate."

Expected behavior

For the check to not require an empty description file (the reported Fail) when a valid ARTICLE.en_us.html is provided

Resources and steps needed to reproduce

See CI run linked above