Merge pull request #1283 from googlefonts/tidy3

Keep .appVersion in .glyphs files
googlefonts · Feb 14, 2025 · 191518c · 191518c
2 parents ba7f319 + a9fe440
commit 191518c
Show file tree

Hide file tree

Showing 88 changed files with 92 additions and 4 deletions.
diff --git a/resources/scripts/tidy_glyphs.py b/resources/scripts/tidy_glyphs.py
@@ -12,10 +12,11 @@
 }
 
 _DELETE_ME = {
-    ".appVersion",
-    "DisplayStrings"
-    "date"
-    "lastChange"
+    # Keep .appVersion, per @anthrotype Glyphs.app makes decisions based on this
+    # ".appVersion",
+    "DisplayStrings",
+    "date",
+    "lastChange",
 }
 
 

diff --git a/resources/testdata/glyphs2/BadIndexing.glyphs b/resources/testdata/glyphs2/BadIndexing.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3148";
 customParameters = (
 {
 name = "Variable Font Origin";

diff --git a/resources/testdata/glyphs2/Component.glyphs b/resources/testdata/glyphs2/Component.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3151";
 familyName = "New Font";
 fontMaster = (
 {

diff --git a/resources/testdata/glyphs2/FeaLegacyName.glyphs b/resources/testdata/glyphs2/FeaLegacyName.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3148";
 familyName = "legacy stylistic set names";
 features = (
 {

diff --git a/resources/testdata/glyphs2/Fea_Class.glyphs b/resources/testdata/glyphs2/Fea_Class.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3148";
 classes = (
 {
 automatic = 1;

diff --git a/resources/testdata/glyphs2/Fea_Feature.glyphs b/resources/testdata/glyphs2/Fea_Feature.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3148";
 customParameters = (
 {
 name = Axes;

diff --git a/resources/testdata/glyphs2/Fea_Order.glyphs b/resources/testdata/glyphs2/Fea_Order.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3148";
 customParameters = (
 {
 name = Axes;

diff --git a/resources/testdata/glyphs2/Fea_Prefix.glyphs b/resources/testdata/glyphs2/Fea_Prefix.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3148";
 customParameters = (
 {
 name = Axes;

diff --git a/resources/testdata/glyphs2/IntermediateLayer.glyphs b/resources/testdata/glyphs2/IntermediateLayer.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3300";
 customParameters = (
 {
 name = "Disable Last Change";

diff --git a/resources/testdata/glyphs2/MVAR.glyphs b/resources/testdata/glyphs2/MVAR.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3228";
 customParameters = (
 {
 name = "Variable Font Origin";

diff --git a/resources/testdata/glyphs2/MasterNames-Italic.glyphs b/resources/testdata/glyphs2/MasterNames-Italic.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3323";
 customParameters = (
 {
 name = "Disable Last Change";

diff --git a/resources/testdata/glyphs2/MasterNames.glyphs b/resources/testdata/glyphs2/MasterNames.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3323";
 customParameters = (
 {
 name = "Disable Last Change";

diff --git a/resources/testdata/glyphs2/MatrixComponent.glyphs b/resources/testdata/glyphs2/MatrixComponent.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3219";
 familyName = MatrixComponent;
 fontMaster = (
 {

diff --git a/resources/testdata/glyphs2/MixedContourComponent.glyphs b/resources/testdata/glyphs2/MixedContourComponent.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3151";
 familyName = "New Font";
 fontMaster = (
 {

diff --git a/resources/testdata/glyphs2/OpszWghtVar_AxisMappings.glyphs b/resources/testdata/glyphs2/OpszWghtVar_AxisMappings.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3148";
 customParameters = (
 {
 name = Axes;

diff --git a/resources/testdata/glyphs2/QCurve.glyphs b/resources/testdata/glyphs2/QCurve.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3208";
 customParameters = (
 {
 name = "Disable Last Change";

diff --git a/resources/testdata/glyphs2/SlantedFont.glyphs b/resources/testdata/glyphs2/SlantedFont.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3227";
 customParameters = (
 {
 name = "Variable Font Origin";

diff --git a/resources/testdata/glyphs2/TheBestNames.glyphs b/resources/testdata/glyphs2/TheBestNames.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3151";
 copyright = "Copy!";
 customParameters = (
 {

diff --git a/resources/testdata/glyphs2/WdthVar.glyphs b/resources/testdata/glyphs2/WdthVar.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3323";
 copyright = "Copy!";
 customParameters = (
 {

diff --git a/resources/testdata/glyphs2/WghtVar.glyphs b/resources/testdata/glyphs2/WghtVar.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3219";
 copyright = "Copy!";
 customParameters = (
 {

diff --git a/resources/testdata/glyphs2/WghtVar.glyphspackage/fontinfo.plist b/resources/testdata/glyphs2/WghtVar.glyphspackage/fontinfo.plist
@@ -1,4 +1,5 @@
 {
+.appVersion = "3219";
 copyright = "Copy!";
 customParameters = (
 {

diff --git a/resources/testdata/glyphs2/WghtVar_Anchors.glyphs b/resources/testdata/glyphs2/WghtVar_Anchors.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3219";
 customParameters = (
 {
 name = Axes;

diff --git a/resources/testdata/glyphs2/WghtVar_Anchors.glyphspackage/fontinfo.plist b/resources/testdata/glyphs2/WghtVar_Anchors.glyphspackage/fontinfo.plist
@@ -1,4 +1,5 @@
 {
+.appVersion = "3219";
 customParameters = (
 {
 name = Axes;

diff --git a/resources/testdata/glyphs2/WghtVar_Avar.glyphs b/resources/testdata/glyphs2/WghtVar_Avar.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3219";
 copyright = "Copy!";
 customParameters = (
 {

diff --git a/resources/testdata/glyphs2/WghtVar_Avar.glyphspackage/fontinfo.plist b/resources/testdata/glyphs2/WghtVar_Avar.glyphspackage/fontinfo.plist
@@ -1,4 +1,5 @@
 {
+.appVersion = "3219";
 copyright = "Copy!";
 customParameters = (
 {

diff --git a/resources/testdata/glyphs2/WghtVar_HeavyHyphen.glyphs b/resources/testdata/glyphs2/WghtVar_HeavyHyphen.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3148";
 customParameters = (
 {
 name = Axes;

diff --git a/resources/testdata/glyphs2/WghtVar_ImplicitAxes.glyphs b/resources/testdata/glyphs2/WghtVar_ImplicitAxes.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3340";
 customParameters = (
 );
 familyName = WghtVar;

diff --git a/resources/testdata/glyphs2/WghtVar_Instances.glyphs b/resources/testdata/glyphs2/WghtVar_Instances.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3219";
 copyright = "Copy!";
 customParameters = (
 {

diff --git a/resources/testdata/glyphs2/WghtVar_Instances.glyphspackage/fontinfo.plist b/resources/testdata/glyphs2/WghtVar_Instances.glyphspackage/fontinfo.plist
@@ -1,4 +1,5 @@
 {
+.appVersion = "3219";
 copyright = "Copy!";
 customParameters = (
 {

diff --git a/resources/testdata/glyphs2/WghtVar_NoExport.glyphs b/resources/testdata/glyphs2/WghtVar_NoExport.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3226";
 copyright = "Copy!";
 customParameters = (
 {

diff --git a/resources/testdata/glyphs2/WghtVar_OS2.glyphs b/resources/testdata/glyphs2/WghtVar_OS2.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3340";
 copyright = "Copy!";
 customParameters = (
 {

diff --git a/resources/testdata/glyphs2/WghtVar_OS2.glyphspackage/fontinfo.plist b/resources/testdata/glyphs2/WghtVar_OS2.glyphspackage/fontinfo.plist
@@ -1,4 +1,5 @@
 {
+.appVersion = "3219";
 copyright = "Copy!";
 customParameters = (
 {

diff --git a/resources/testdata/glyphs2/alignment_zones_v2.glyphs b/resources/testdata/glyphs2/alignment_zones_v2.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "1365";
 familyName = "New Font";
 fontMaster = (
 {

diff --git a/resources/testdata/glyphs2/infinity.glyphs b/resources/testdata/glyphs2/infinity.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3219";
 customParameters = (
 {
 name = Axes;

diff --git a/resources/testdata/glyphs2/infinity.glyphspackage/fontinfo.plist b/resources/testdata/glyphs2/infinity.glyphspackage/fontinfo.plist
@@ -1,4 +1,5 @@
 {
+.appVersion = "3219";
 customParameters = (
 {
 name = Axes;

diff --git a/resources/testdata/glyphs3/Component.glyphs b/resources/testdata/glyphs3/Component.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3151";
 .formatVersion = 3;
 familyName = "New Font";
 fontMaster = (

diff --git a/resources/testdata/glyphs3/ComponentAnchor.glyphs b/resources/testdata/glyphs3/ComponentAnchor.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3259";
 .formatVersion = 3;
 familyName = "New Font";
 fontMaster = (

diff --git a/resources/testdata/glyphs3/CustomOrigin.glyphs b/resources/testdata/glyphs3/CustomOrigin.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3227";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/EmptyMetaTable.glyphs b/resources/testdata/glyphs3/EmptyMetaTable.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3260";
 .formatVersion = 3;
 customParameters = (
 {

diff --git a/resources/testdata/glyphs3/Fea_Labels.glyphs b/resources/testdata/glyphs3/Fea_Labels.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3317";
 .formatVersion = 3;
 familyName = "GlobalMetrics Font Test";
 features = (

diff --git a/resources/testdata/glyphs3/FixedPitch.glyphs b/resources/testdata/glyphs3/FixedPitch.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3219";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/GlobalMetrics_font_customParameters.glyphs b/resources/testdata/glyphs3/GlobalMetrics_font_customParameters.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3316";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/IntermediateLayer.glyphs b/resources/testdata/glyphs3/IntermediateLayer.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3300";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/InvalidVendorID.glyphs b/resources/testdata/glyphs3/InvalidVendorID.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3219";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/MVAR.glyphs b/resources/testdata/glyphs3/MVAR.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3228";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/MasterNames-Italic.glyphs b/resources/testdata/glyphs3/MasterNames-Italic.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3323";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/MasterNames.glyphs b/resources/testdata/glyphs3/MasterNames.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3323";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/MetaTable.glyphs b/resources/testdata/glyphs3/MetaTable.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3260";
 .formatVersion = 3;
 customParameters = (
 {

diff --git a/resources/testdata/glyphs3/NestedComponent.glyphs b/resources/testdata/glyphs3/NestedComponent.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3151";
 .formatVersion = 3;
 customParameters = (
 {

diff --git a/resources/testdata/glyphs3/NoMetaTable.glyphs b/resources/testdata/glyphs3/NoMetaTable.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3260";
 .formatVersion = 3;
 familyName = NoMetaTable;
 fontMaster = (

diff --git a/resources/testdata/glyphs3/Oswald-AE-comb.glyphs b/resources/testdata/glyphs3/Oswald-AE-comb.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3223";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/Oswald-O.glyphs b/resources/testdata/glyphs3/Oswald-O.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3192";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/Oswald-glyph-categories.glyphs b/resources/testdata/glyphs3/Oswald-glyph-categories.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3260";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/PropagateAnchorsTest-propagated.glyphs b/resources/testdata/glyphs3/PropagateAnchorsTest-propagated.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3300";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/PropagateAnchorsTest.glyphs b/resources/testdata/glyphs3/PropagateAnchorsTest.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3300";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/QCurve.glyphs b/resources/testdata/glyphs3/QCurve.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3208";
 .formatVersion = 3;
 customParameters = (
 {

diff --git a/resources/testdata/glyphs3/SlantedFont.glyphs b/resources/testdata/glyphs3/SlantedFont.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3227";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/StandardAxisNames.glyphs b/resources/testdata/glyphs3/StandardAxisNames.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3324";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/StaticBoldItalic.glyphs b/resources/testdata/glyphs3/StaticBoldItalic.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3151";
 .formatVersion = 3;
 familyName = WghtVar;
 fontMaster = (

diff --git a/resources/testdata/glyphs3/StaticWithInstance.glyphs b/resources/testdata/glyphs3/StaticWithInstance.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3151";
 .formatVersion = 3;
 familyName = WghtVar;
 fontMaster = (

diff --git a/resources/testdata/glyphs3/TheBestNames.glyphs b/resources/testdata/glyphs3/TheBestNames.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3151";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/TheBestV2NamesInAV3File.glyphs b/resources/testdata/glyphs3/TheBestV2NamesInAV3File.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3151";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/UnusualCustomParams.glyphs b/resources/testdata/glyphs3/UnusualCustomParams.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3260";
 .formatVersion = 3;
 customParameters = (
 {

diff --git a/resources/testdata/glyphs3/WdthVar.glyphs b/resources/testdata/glyphs3/WdthVar.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3316";
 .formatVersion = 3;
 axes = (
 {

diff --git a/resources/testdata/glyphs3/WghtVar.glyphs b/resources/testdata/glyphs3/WghtVar.glyphs
@@ -1,4 +1,5 @@
 {
+.appVersion = "3219";
 .formatVersion = 3;
 axes = (
 {