Add rationale for metadata/license

com.google.fonts/check/metadata/license On the Google Fonts Profile. (PR #4570)
fonttools · Mar 5, 2024 · ac87f85 · ac87f85
1 parent 51856ab
commit ac87f85
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -52,6 +52,7 @@ A more detailed list of changes is available in the corresponding milestones for
   - **[com.google.fonts/check/production_glyphs_similarity]:** Added rationale. (PR #4570)
   - **[com.google.fonts/check/version_bump]:** Added rationale. (PR #4570)
   - **[com.google.fonts/check/family/has_license]:** Added rationale. (PR #4570)
+  - **[com.google.fonts/check/metadata/license]:** Added rationale. (PR #4570)
 
 #### On the FontValidator profile
   - **[com.google.fonts/check/fontvalidator]:** Added rationale. (PR #4570)

diff --git a/Lib/fontbakery/checks/googlefonts/license.py b/Lib/fontbakery/checks/googlefonts/license.py
@@ -448,6 +448,11 @@ def com_google_fonts_check_name_license_url(ttFont, familyname):
     id="com.google.fonts/check/metadata/license",
     conditions=["family_metadata"],
     proposal="legacy:check/085",
+    rationale="""
+        The license field in METADATA.pb must contain one of the
+        three values "APACHE2", "UFL" or "OFL". (New fonts should
+        generally be OFL unless there are special circumstances.)
+    """,
 )
 def com_google_fonts_check_metadata_license(family_metadata):
     """METADATA.pb license is "APACHE2", "UFL" or "OFL"?"""