Reem Kufi Fun: Version 1.005 added

[yanone]

9239ee3: [gftools-packager] Reem Kufi Fun: Version 1.005 added

• Reem Kufi Fun Version 1.005 taken from the upstream repo https://github.com/aliftype/reem-kufi at commit https://github.com/aliftype/reem-kufi/commit/.

[gf-bot]

Fontbakery report

Fontbakery version: 0.8.10

[1] Family checks

ℹ INFO: Check axis ordering on the STAT table. (com.google.fonts/check/STAT/axis_order)

• ℹ INFO From a total of 1 font files, 0 of them (0.00%) lack a STAT table.

  And these are the most common STAT axis orderings:
  ('wght', 1) [code: summary]

[18] ReemKufiFun[wght].ttf

🔥 FAIL: Check METADATA.pb parse correctly. (com.google.fonts/check/metadata/parses)

• 🔥 FAIL Family metadata at ofl/reemkufifun failed to parse.
  TRACEBACK:
  29:1 : Message type "google.fonts_public.FamilyProto" has no field named "primary_script". [code: parsing-error]

🔥 FAIL: Check font names are correct (com.google.fonts/check/font_names)

• 🔥 FAIL Font names are incorrect:

nameID	current	expected
Family Name	Reem Kufi Fun	Reem Kufi Fun
Subfamily Name	Regular	Regular
Full Name	Reem Kufi Fun	Reem Kufi Fun Regular
Poscript Name	ReemKufiFun	ReemKufiFun-Regular

• ⚠ WARN Regular missing from full name [code: lacks-regular]

🔥 FAIL: Font enables smart dropout control in "prep" table instructions? (com.google.fonts/check/smart_dropout)

• 🔥 FAIL The 'prep' table does not contain TrueType instructions enabling smart dropout control. To fix, export the font with autohinting enabled, or run ttfautohint on the font, or run the gftools fix-nonhinting script. [code: lacks-smart-dropout]

🔥 FAIL: Glyph names are all valid? (com.google.fonts/check/valid_glyphnames)

• 🔥 FAIL The following glyph names do not comply with naming conventions: hamza-ar, highhamza-ar, alef-ar, alef-ar.1, alef-ar.2, alef-ar.3, alef-ar.fina, alef-ar.fina.Lam, alef-ar.fina.Lam.1, behDotless-ar and 219 more.

Use -F or --full-lists to disable shortening of long lists.

A glyph name must be entirely comprised of characters from the following set: A-Z a-z 0-9 .(period) (underscore). A glyph name must not start with a digit or period. There are a few exceptions such as the special glyph ".notdef". The glyph names "twocents", "a1", and "" are all valid, while "2cents" and ".twocents" are not. [code: found-invalid-names]

🔥 FAIL: Validates that when an instance record is included for the default instance, its subfamilyNameID value is set to either 2 or 17 (or something with the same value as 2), and its postScriptNameID value is set to 6 (or something with the same value as 6). (com.adobe.fonts/check/varfont/valid_default_instance_nameids)

• 🔥 FAIL 'Regular' instance has the same coordinates as the default instance; its postscript name should be 'ReemKufiFun', instead of 'ReemKufiFun-Regular'. [code: invalid-default-instance-postscript-nameid:261]

⚠ WARN: Description strings in the name table must not exceed 200 characters. (com.google.fonts/check/name/description_max_length)

• ⚠ WARN A few name table entries with ID=10 (NameID.DESCRIPTION) are longer than 200 characters. Please check whether those entries are copyright notices mistakenly stored in the description string entries by a bug in an old FontLab version. If that's the case, then such copyright notices must be removed from these entries. [code: too-long]

⚠ WARN: A static fonts directory with at least two fonts must accompany variable fonts (com.google.fonts/check/repo/vf_has_static_fonts)

• ⚠ WARN Please consider adding a subdirectory called "static/" and including in it static font files. [code: missing]

⚠ WARN: Ensure variable fonts include an avar table. (com.google.fonts/check/mandatory_avar_table)

• ⚠ WARN This variable font does not have an avar table. [code: missing-avar]

⚠ WARN: Ensure fonts have ScriptLangTags declared on the 'meta' table. (com.google.fonts/check/meta/script_lang_tags)

• ⚠ WARN This font file does not have a 'meta' table. [code: lacks-meta-table]

⚠ WARN: Check font contains no unreachable glyphs (com.google.fonts/check/unreachable_glyphs)

• ⚠ WARN The following glyphs could not be reached by codepoint or substitution rules:

  • IJacute

  • ijacute

  • macronbelowcomba

  • threedotsdownabove-ar

  • And threedotsupbelow-ar
    [code: unreachable-glyphs]

ℹ INFO: Does DESCRIPTION file contain a upstream Git repo URL? (com.google.fonts/check/description/git_url)

• ℹ INFO Found a git repo URL: https://github.com/alif-type/reem-kufi [code: url-found]

ℹ INFO: Show hinting filesize impact. (com.google.fonts/check/hinting_impact)

• ℹ INFO Hinting filesize impact:

	ofl/reemkufifun/ReemKufiFun[wght].ttf
Dehinted Size	139.4kb
Hinted Size	139.4kb
Increase	0 bytes
Change	0.0 %
[code: size-impact]

ℹ INFO: Font has old ttfautohint applied? (com.google.fonts/check/old_ttfautohint)

• ℹ INFO Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 1.005'] [code: version-not-detected]

ℹ INFO: EPAR table present in font? (com.google.fonts/check/epar)

• ℹ INFO EPAR table not present in font. To learn more see When missing EPAR table, just emit INFO alert fonttools/fontbakery#818 [code: lacks-EPAR]

ℹ INFO: Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? (com.google.fonts/check/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]

ℹ INFO: Check for font-v versioning. (com.google.fonts/check/fontv)

• ℹ INFO Version string is: "Version 1.005"
  The version string must ideally include a git commit hash and either a "dev" or a "release" suffix such as in the example below:
  "Version 1.3; git-0d08353-release" [code: bad-format]

ℹ INFO: Font contains all required tables? (com.google.fonts/check/required_tables)

• ℹ INFO This font contains the following optional tables:

  • loca

  • GPOS

  • GSUB

  • And gasp [code: optional-tables]

ℹ INFO: List all superfamily filepaths (com.google.fonts/check/superfamily/list)

• ℹ INFO ofl/reemkufifun [code: family-path]

### Summary

💔 ERROR	🔥 FAIL	⚠ WARN	💤 SKIP	ℹ INFO	🍞 PASS	🔎 DEBUG
0	5	5	91	9	117	0
0%	2%	2%	40%	4%	52%	0%

Note: The following loglevels were omitted in this report:

• SKIP
• PASS
• DEBUG

[RosaWagner]

Since Rod reported a missing SVG table in Foldit, I checked this one for it manually, and there is none.

[m4rc1e]

The FB check which tests this still needs merging fonttools/fontbakery#3889

[yanone]

I'll let Khaled know so he can integrate maximum_color it into his repo

[yanone]

Khaled says the expected "Regular" in the name checks are nonsense, as this is a Regular font. Will this be able to pass?

[m4rc1e]

It's fine since previous version was like this. I'm going to just raise a WARN if Regular fonts are missing Regular of the fullname and ps name in the future.

[yanone]

Another quite important question: Khaled just verified why I ask a SVG table of him given that this is a COLRv0 font. I think he's right. The fontbakery check is explicit about that and actually complains when a COLRv0 font has a SVG table, while it demands one for a COLRv1 font.

[m4rc1e]

Correct, COLR v0 doesn't need it. However, the FB check for this has only just been merged fonttools/fontbakery#3889 so we'll only be able to test this correctly once the next release is out.

[yanone]

So then this one is good to go