laerm0 / opensans

Open Sans font
Apache License 2.0
3 stars 3 forks source link

Normalize UFO and document how to do so #11

Closed moyogo closed 5 years ago

moyogo commented 5 years ago

Normalizing UFOs can make commits smaller and diffs more relevant. As mentioned in #10, you can either use a tool like ufoNormalizer or fontTools.ufoLib, or also psfnormalize.

Whichever one you pick, please document it so any contributor can use the same.

laerm0 commented 5 years ago

Hi @moyogo –

Based on @anthrotype's input at https://github.com/laerm0/opensans/pull/10#issuecomment-491310557, I don't think I need to normalize. The build path here will use fontTools.ufoLib for the foreseeable future, so I'll stick with what I know unless there's a pressing reason to change (i.e. if this switches from Glyphs to UFO source).

moyogo commented 5 years ago

Please indicate this in the README.md. I’m just thinking of the case where someone submits a merge request made with another UFO writer where the normalization is different.

laerm0 commented 5 years ago

Excellent point. I will do so.

laerm0 commented 5 years ago

Closing as README is now updated.