ryanoasis / nerd-fonts

Iconic font aggregator, collection, & patcher. 3,600+ icons, 50+ patched fonts: Hack, Source Code Pro, more. Glyph collections: Font Awesome, Material Design Icons, Octicons, & more
https://NerdFonts.com
Other
52.86k stars 3.6k forks source link

Update all Iosevka variants to version 29.0.4 #1497

Closed iruoy closed 3 months ago

iruoy commented 5 months ago

Description

Bring all Iosevka versions up to date

Requirements / Checklist

What does this Pull Request (PR) do?

How should this be manually tested?

Any background context you can provide?

What are the relevant tickets (if any)?

Screenshots (if appropriate or helpful)

Finii commented 5 months ago

Thanks for the nice PR!

I would put this on hold until directly before the Nerd Fonts v3.2 release, as there might be another Iosevka release in between.

And then one question: You put all the weights into individual directories for IosevkaTermSlab, where it was one big flat directory before. Is there a reason for that, except because the other Iosevkas have it and you wanted to unify?

Do you download the fonts from the website for installation?

The flat setup in the newer variant - do you see advantages in the 'deep' setup?

My reasoning was that grouping by weight is not very useful, as people usually want all weights but only one variant (Mono, Propo). Did not finish that redesign, but that is where the flat source (and patched) directories come from.

What I mean:

.../patched_fonts/
                  Iosevka/
                          Nerd Font/         (all the weights, flat)
                          Nerd Font Mono/    (all weights, flat)
                          Nerd Font Propo/   (you get the idea)

In the release packages they are always flat anyhow.

iruoy commented 5 months ago

Hey, thanks for your extensive comment.

First off, I think it's a good idea to hold off on merging this until right before a new version is tagged, because Iosevka releases so often. There's even been a new release since I made this PR.

The reason for the changes to IosevkaTermSlab was indeed to synchronize the workflow across the three Iosevka folders. I still think that's a good thing to do. Apparently, I settled on the wrong concept though. Tomorrow, I'll try to wrangle these configs into generating the folder structure you're looking for.

Finii commented 5 months ago

I still think that's a good thing to do.

What do you mean by 'it'? Unifying or having a deep-weights-directory-structure? If the second, can you explain? Just because I once thought it (*) would be a good idea does not make it the best solution. Every discussion welcome.

Edit: (*) This it means a-flat-structure

Finii commented 3 months ago

Updated this PR to use 29.0.4 which is current. Also drop patched fonts, these are created by the release process. Also some small other changes.

Finii commented 3 months ago

@allcontributors please add @iruoy for code

allcontributors[bot] commented 3 months ago

@Finii

I've put up a pull request to add @iruoy! :tada: