Skip to content
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

Move render_own_name back to Google Fonts profile #4951

Merged
merged 1 commit into from
Dec 29, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@


@check(
id="render_own_name",
id="googlefonts/render_own_name",
rationale="""
A base expectation is that a font family's regular/default (400 roman) style
can render its 'menu name' (nameID 1) in itself.
Expand Down
2 changes: 1 addition & 1 deletion Lib/fontbakery/legacy_checkids.py
Original file line number Diff line number Diff line change
Expand Up @@ -130,6 +130,7 @@
"com.google.fonts/check/old_ttfautohint": "googlefonts/old_ttfautohint",
"com.google.fonts/check/production_encoded_glyphs": "googlefonts/production_encoded_glyphs",
"com.google.fonts/check/production_glyphs_similarity": "googlefonts/production_glyphs_similarity",
"com.google.fonts/check/render_own_name": "googlefonts/render_own_name",
"com.google.fonts/check/repo/dirname_matches_nameid_1": "googlefonts/repo/dirname_matches_nameid_1",
"com.google.fonts/check/repo/fb_report": "googlefonts/repo/fb_report",
"com.google.fonts/check/repo/sample_image": "googlefonts/repo/sample_image",
Expand Down Expand Up @@ -273,7 +274,6 @@
"com.google.fonts/check/outline_jaggy_segments": "outline_jaggy_segments",
"com.google.fonts/check/outline_semi_vertical": "outline_semi_vertical",
"com.google.fonts/check/outline_short_segments": "outline_short_segments",
"com.google.fonts/check/render_own_name": "render_own_name",
"com.google.fonts/check/required_tables": "required_tables",
"com.google.fonts/check/rupee": "rupee",
"com.adobe.fonts/check/sfnt_version": "sfnt_version",
Expand Down
1 change: 0 additions & 1 deletion Lib/fontbakery/profiles/adobefonts.py
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,6 @@
"no_debugging_tables",
"no_mac_entries",
"overlapping_path_segments",
"render_own_name",
"smallcaps_before_ligatures",
"smart_dropout",
"soft_hyphen",
Expand Down
1 change: 0 additions & 1 deletion Lib/fontbakery/profiles/fontbureau.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@
"nested_components",
"no_mac_entries",
"overlapping_path_segments",
"render_own_name",
"smart_dropout",
"stylisticset_description",
"typographic_family_name",
Expand Down
1 change: 1 addition & 0 deletions Lib/fontbakery/profiles/googlefonts.py
Original file line number Diff line number Diff line change
Expand Up @@ -138,6 +138,7 @@
"googlefonts/old_ttfautohint",
"googlefonts/production_glyphs_similarity",
# "googlefonts/production_encoded_glyphs", # DISABLED
"googlefonts/render_own_name",
"googlefonts/STAT/axis_order",
"googlefonts/STAT/axisregistry",
"googlefonts/STAT/compulsory_axis_values",
Expand Down
1 change: 0 additions & 1 deletion Lib/fontbakery/profiles/microsoft.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@
"no_debugging_tables",
"no_mac_entries",
"overlapping_path_segments",
"render_own_name",
"smallcaps_before_ligatures",
"smart_dropout",
"stylisticset_description",
Expand Down
2 changes: 1 addition & 1 deletion Lib/fontbakery/profiles/notofonts.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,8 @@
"exclude_checks": [
"googlefonts/glyph_coverage",
"googlefonts/gasp",
"googlefonts/render_own_name",
#
"render_own_name",
"smart_dropout",
],
"configuration_defaults": {
Expand Down
1 change: 0 additions & 1 deletion Lib/fontbakery/profiles/typenetwork.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@
"math_signs_width", # It really depends on the design and the intended use to make math symbols the same width.
"name/no_copyright_on_description",
"os2_metrics_match_hhea", # Removed in favor of new vmetrics check
"render_own_name",
"STAT_strings", # replaced by adobefonts/STAT_strings
"superfamily/list",
"ufolint", # < TypeNetwork doesn’t check .ufo files.
Expand Down
1 change: 0 additions & 1 deletion Lib/fontbakery/profiles/universal.py
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,6 @@
"ots",
"overlapping_path_segments",
"required_tables",
"render_own_name",
"rupee",
"sfnt_version",
"smallcaps_before_ligatures",
Expand Down
11 changes: 0 additions & 11 deletions tests/test_checks_glyphset.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,17 +19,6 @@ def test_can_shape():
assert not can_shape(font, "こんにちは")


@check_id("render_own_name")
def test_check_render_own_name(check):
"""Check family directory name."""

ttFont = TEST_FILE("overpassmono/OverpassMono-Regular.ttf")
assert_PASS(check(ttFont))

ttFont = TEST_FILE("noto_sans_tamil_supplement/NotoSansTamilSupplement-Regular.ttf")
assert_results_contain(check(ttFont), FAIL, "render-own-name")


@check_id("control_chars")
def test_check_family_control_chars(check):
"""Are any unacceptable control characters present in font files?"""
Expand Down
18 changes: 18 additions & 0 deletions tests/test_checks_googlefonts_render_own_name.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
from conftest import check_id
from fontbakery.codetesting import (
assert_PASS,
assert_results_contain,
TEST_FILE,
)
from fontbakery.status import FAIL


@check_id("googlefonts/render_own_name")
def test_check_render_own_name(check):
"""Check family directory name."""

ttFont = TEST_FILE("overpassmono/OverpassMono-Regular.ttf")
assert_PASS(check(ttFont))

ttFont = TEST_FILE("noto_sans_tamil_supplement/NotoSansTamilSupplement-Regular.ttf")
assert_results_contain(check(ttFont), FAIL, "render-own-name")
Loading