Steven R. Loomis
Steven R. Loomis
@macchiati merging this year-old PR that makes structural improvements. I'll tackle the message in a separate PR.
lots of good deletion here! For the BOMs, would it make sense just to fix them up with a build step?
@markusicu LGTM ready to merge?
> More specifically, the data comes from ICU (a dependency of V8). even more specifically, ICU's data comes from [cldr](https://github.com/unicode-org/cldr), a dependency of ICU
> cc @srl295 in case you missed this pull request I missed it, but it's [fixed upstream](https://github.com/nodejs/node/issues/50498#issuecomment-1789597049)
Absence of md5 is an ICU issue and was fixed, However verifying with gpg is not a bad idea.
@macchiati ok to merge this?
I don't have a good example. Any general caching mechanism could work.
> Has this been pushed to staging for UI review? Yes and you reviewed it, i'm preparing another push incorporating your feedback (such as the word 'inherited')