googlefonts / googlefonts/fontheight

`static-lang-word-lists` testing: ensure script & language are recognised by HarfBuzz

Open
#55 0 comments 0 reactions 0 assignees View on GitHub
static-lang-word-lists
Dominant language
Rust
Stars
9
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Pull in `harfrust` as a dev dependency and smash all the word lists' metadata through:

- `Tag::from_str` + `Script::from_iso15924_tag` for script
- `Language::from_str` for language

See the `impl WordListExt for WordList` code in Font Height to see how things are checked there

We won't be able to remove that validation from Font Height as we may eventually add runtime-defined word lists which don't necessarily need to have HarfBuzz recognised scripts/languages (and pulling in `harfrust` as a runtime dependency to be able to check that feels very big & overkill)

This should be implementable as an integration test (and therefore without codegen 🙌)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.