nerd-fonts icon indicating copy to clipboard operation
nerd-fonts copied to clipboard

Bugfix/ci release run

Open Finii opened this issue 2 years ago • 2 comments

Description

Fix release CI pipeline that does not patch all fonts.

Requirements / Checklist

What does this Pull Request (PR) do?

Allow the release script to also patch fonts where the directory name of the font in src/ is not the same as the (beginning of the ) font file name.

How should this be manually tested?

Let the CI run and examine the job. All sub-jobs from the matrix have to succeed and that means now that they patch at least one font file. Afterwards examine that all fonts in patched-fonts/ have been rebuild.

Any background context you can provide?

What are the relevant tickets (if any)?

#824

Please also consider related #786

Screenshots (if appropriate or helpful)

Finii avatar Apr 22 '22 12:04 Finii

LGTM

After looking in their Contribution activity timeline ... Leaving 'LGTM' in random repos to get the reviews count up?

Finii avatar Apr 29 '22 07:04 Finii

I only review things I think I know about.

kamack38 avatar Apr 29 '22 10:04 kamack38