Get wasm building

Author: simoncozens

profile-googlefonts/src/checks/googlefonts/font_copyright.rs

@@ -2,8 +2,7 @@ use fontspector_checkapi::{prelude::*, FileTypeConvert};
 use hashbrown::HashMap;
 use skrifa::string::StringId;
 
-use crate::checks::googlefonts::metadata::family_proto;
-use crate::constants::EXPECTED_COPYRIGHT_PATTERN;
+use crate::{checks::googlefonts::metadata::family_proto, constants::EXPECTED_COPYRIGHT_PATTERN};
 
 #[check(
     id = "googlefonts/font_copyright",
@@ -71,7 +70,7 @@ fn font_copyright(c: &TestableCollection, context: &Context) -> CheckFnResult {
             .file_name()
             .and_then(|x| x.to_str())
             .unwrap_or_default();
-        let filename = font.filename.as_os_str().to_str().unwrap_or_default();
+        // let filename = font.filename.as_os_str().to_str().unwrap_or_default();
         let mut copyrights = vec![];
         font.get_name_entry_strings(StringId::COPYRIGHT_NOTICE)
             .for_each(|x| copyrights.push(("Name Table entry", x)));

profile-googlefonts/src/checks/googlefonts/metadata/mod.rs

@@ -29,6 +29,7 @@ mod protos {
     #![allow(clippy::all, clippy::unwrap_used)]
     include!(concat!(env!("OUT_DIR"), "/protos/mod.rs"));
 }
+#[allow(unused_imports)]
 pub(crate) use designers::DesignerInfoProto;
 pub(crate) use fonts_public::FamilyProto;
 use fontspector_checkapi::{CheckError, Testable};
@@ -51,5 +52,7 @@ mod consistent_repo_urls;
 pub use consistent_repo_urls::consistent_repo_urls;
 mod primary_script;
 pub use primary_script::primary_script;
+#[cfg(not(target_family = "wasm"))]
 mod designer_profiles;
+#[cfg(not(target_family = "wasm"))]
 pub use designer_profiles::designer_profiles;

profile-googlefonts/src/lib.rs

@@ -48,17 +48,21 @@ impl fontspector_checkapi::Plugin for GoogleFonts {
             .add_and_register_check(checks::googlefonts::metadata::category)
             //            checks::googlefonts::metadata::category_hints // merged into metadata/validate
             .add_and_register_check(checks::googlefonts::metadata::consistent_repo_urls)
-            .add_and_register_check(checks::googlefonts::metadata::consistent_with_fonts)
-            //            checks::googlefonts::metadata::filenames // merged into metadata/consistent_with_fonts
-            //            checks::googlefonts::metadata::canonical_style_names // merged into metadata/consistent_with_fonts
-            //            checks::googlefonts::metadata::valid_full_name_values // merged into metadata/consistent_with_fonts
-            //            checks::googlefonts::metadata::nameid/post_script_name // merged into metadata/consistent_with_fonts
-            //            checks::googlefonts::metadata::valid_post_script_name_values // merged into metadata/consistent_with_fonts
-            //            checks::googlefonts::metadata::nameid/family_and_full_names // merged into metadata/consistent_with_fonts
-            //            checks::googlefonts::metadata::valid_filename_values // redundant, see fontbakery#4997
-            //            checks::googlefonts::metadata::undeclared_fonts // redundant, see fontbakery#4997
-            //            checks::googlefonts::metadata::nameid/font_name // redundant, see fontbakery#4581
-            .add_and_register_check(checks::googlefonts::metadata::designer_profiles)
+            .add_and_register_check(checks::googlefonts::metadata::consistent_with_fonts);
+        //            checks::googlefonts::metadata::filenames // merged into metadata/consistent_with_fonts
+        //            checks::googlefonts::metadata::canonical_style_names // merged into metadata/consistent_with_fonts
+        //            checks::googlefonts::metadata::valid_full_name_values // merged into metadata/consistent_with_fonts
+        //            checks::googlefonts::metadata::nameid/post_script_name // merged into metadata/consistent_with_fonts
+        //            checks::googlefonts::metadata::valid_post_script_name_values // merged into metadata/consistent_with_fonts
+        //            checks::googlefonts::metadata::nameid/family_and_full_names // merged into metadata/consistent_with_fonts
+        //            checks::googlefonts::metadata::valid_filename_values // redundant, see fontbakery#4997
+        //            checks::googlefonts::metadata::undeclared_fonts // redundant, see fontbakery#4997
+        //            checks::googlefonts::metadata::nameid/font_name // redundant, see fontbakery#4581
+        #[cfg(not(target_family = "wasm"))]
+        let builder =
+            builder.add_and_register_check(checks::googlefonts::metadata::designer_profiles);
+
+        let builder = builder
             .add_and_register_check(checks::googlefonts::metadata::escaped_strings)
             //            .add_and_register_check(checks::googlefonts::metadata::family_directory_name)
             .add_and_register_check(checks::googlefonts::metadata::familyname)

profile-googlefonts/src/network_conditions.rs

@@ -181,6 +181,7 @@ pub(crate) fn get_url(
     request.send().and_then(|r| r.error_for_status())
 }
 
+#[allow(dead_code)]
 pub const DESIGNER_INFO_RAW_URL: &str =
     "https://raw.githubusercontent.com/google/fonts/master/catalog/designers/";