From 45c432fe0d16b2e96e6761ade3b0feaa043ee7c7 Mon Sep 17 00:00:00 2001 From: Fini Jastrow Date: Tue, 18 Jun 2024 08:28:04 +0200 Subject: [PATCH] doc: State that no patched fonts are to be manually committed [why] It is explicitely explained for updating fonts but not for new fonts. [how] Copy appropriate text passage to new fonts paragraph. Fixes: #1655 Signed-off-by: Fini Jastrow --- contributing.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/contributing.md b/contributing.md index 17270be1d..274b198d0 100644 --- a/contributing.md +++ b/contributing.md @@ -64,6 +64,8 @@ This is not needed and you should never commit any patched files directly to the * `fontforge --script ./font-patcher src/unpatched-fonts/XYZ/XYZ.ttf --complete --debug 2` * Make sure to then delete this new font file if it is in the repository (all patched fonts should be generated in the `patched-fonts` directory) ### 4. Run build scripts +You should never commit any patched files directly to the repo. The Github workflow will do that at appropriate times. + * When fairly satisfied the font patches correctly, run the following: * Patch **all** of the variations/options, e.g. * `NERDFONTS='--debug 2 --makegroups 1' ./gotta-patch-em-all-font-patcher\!.sh /XYZ`