googlefonts/googlefonts-project-template

Add `fontbakery --full-lists` to makefile

Closed this issue · 3 comments

When I work on a font that uses googlefonts-project-template, I often read the Fontbakery report in the actions tab in the GitHub UI. I suspect many other template users probably do the same thing.

There are some Fontbakery checks with long lists that get truncated, specifically com.google.fonts/check/glyph_coverage.

I usually make a PR adding fontbakery --full-lists to the make file so I can see the full list: ladinoprojects/solitreo#17

This isn't a great user experience, for some less technical users, adding the --full-lists flag might be too intimidating or confusing.

I suggest making fontbakery --full-lists the default behavior.

I have used the modified command below and it works.

test: venv build.stamp
	. venv/bin/activate; mkdir -p out/ out/fontbakery; fontbakery check-googlefonts -l WARN --full-lists --succinct --badges out/badges --html out/fontbakery/fontbakery-report.html --ghmarkdown out/fontbakery/fontbakery-report.md $(shell find fonts/ttf -type f)

Before fontbakery --full-lists
before

After fontbakery --full-lists
after

Note that Fontbakery says nothing about the makefile from the Google Fonts Project Template and assumes you are running Fontbakery from the command line.

Somewhat duplicate issue here: #90

This was fixed by PR #92