Steven R. Loomis

Results 747 comments of 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.

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')