-
Notifications
You must be signed in to change notification settings - Fork 2.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pochaevsk: Version 1.210; ttfautohint (v1.8.4.7-5d5b) added #9016
Conversation
Taken from the upstream repo https://github.com/slavonic/pochaevsk at commit slavonic/Pochaevsk@4238da8. Resolves #8727
Author just completed the GF_Cyrillic_Historical glyphset so that the font can show up for "Church Slavic" language setting in the GF UI. |
FontBakery reportfontbakery version: 0.13.1 Experimental checksThese won't break the CI job for now, but will become effective after some time if nobody raises any concern. [1] Pochaevsk-Regular.ttf💥 ERROR Check base characters have non-zero advance width. base_has_width
[code: failed-check] All other checks[26] Pochaevsk-Regular.ttf🔥 FAIL METADATA.pb: Check URL on copyright string is the same as in repository_url field. googlefonts/metadata/consistent_repo_urls
But: OFL text has 'https://github.com/slavonic/Pochaevsk' [code: mismatch]🔥 FAIL Check Google Fonts glyph coverage. googlefonts/glyph_coverage
[code: missing-codepoints] 🔥 FAIL Check if the vertical metrics of a family are similar to the same family hosted on Google Fonts. googlefonts/vertical_metrics_regressions
|
ofl/pochaevsk/Pochaevsk-Regular.ttf | |
---|---|
Dehinted Size | 115.1kb |
Hinted Size | 175.9kb |
Increase | 60.8kb |
Change | 52.9 % |
ℹ️ INFO Font contains all required tables? required_tables
- ℹ️ INFO
This font contains the following optional tables:
- cvt
- fpgm
- loca
- prep
- GPOS
- GSUB
- gasp
[code: optional-tables]
ℹ️ INFO METADATA.pb: Validate family.minisite_url field. googlefonts/metadata/minisite_url
- ℹ️ INFO
Please consider adding a family.minisite_url entry.
[code: lacks-minisite-url]
ℹ️ INFO Does DESCRIPTION file contain a upstream Git repo URL? googlefonts/description/git_url
- ℹ️ INFO
Found a git repo URL: https://github.com/slavonic/pochaevsk
[code: url-found]
ℹ️ INFO Check for presence of an ARTICLE.en_us.html file googlefonts/description/has_article
- ℹ️ INFO
This font doesn't have an ARTICLE.en_us.html file.
[code: missing-article]
ℹ️ INFO Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? googlefonts/gasp
- ℹ️ INFO
These are the ppm ranges declared on the gasp table:
PPM <= 65535: flag = 0x0F - Use grid-fitting - Use grayscale rendering - Use gridfitting with ClearType symmetric smoothing - Use smoothing along multiple axes with ClearType®
[code: ranges][1] Family checks
ℹ️ INFO Check axis ordering on the STAT table. googlefonts/STAT/axis_order
- ℹ️ INFO
All of the fonts lack a STAT table.
[code: summary]
Summary
💥 ERROR | ☠ FATAL | 🔥 FAIL | ⏩ SKIP | ℹ️ INFO | ✅ PASS | 🔎 DEBUG | |
---|---|---|---|---|---|---|---|
1 | 0 | 3 | 16 | 54 | 8 | 154 | 0 |
0% | 0% | 1% | 7% | 23% | 3% | 65% | 0% |
Note: The following loglevels were omitted in this report:
- SKIP
- PASS
- DEBUG
I don't understand the vertical metrics FAIL. The font currently live on GF has a typo/hhea descender of -400. |
Taken from the upstream repo https://github.com/slavonic/pochaevsk at commit slavonic/Pochaevsk@4238da8.
Resolves #8727
PR Checklist:
primary_script
definition in the METADATA.pb file for all projects that have a primary non-Latin based language support targetsubsets
definitions in the METADATA.pb reflect the actual subsets and languages present in the font files (in alphabetic order). For CJK fonts, only include one of the following subsetschinese-hongkong
,chinese-simplified
,chinese-traditional
,korean
,japanese
.