search

unicode-org / icu4x

Solving i18n for client-side and resource-constrained environments.
https://icu4x.unicode.org
Other
1.38k stars 178 forks source link

Remove ResolvedNeoTimeZoneSkeleton #5774

Closed robertbastian closed 2 weeks ago

robertbastian commented 2 weeks ago

This is blocking #5773

sffc commented 2 weeks ago

Here is the post where I compared and contrasted the different approaches to the time zone intermediate type. There was a somewhat involved discussion with @Manishearth, @zbraniecki, and myself.

https://github.com/unicode-org/icu4x/pull/5024#issuecomment-2157351593

I had implemented option 2ii. This PR basically reverts us back to option 1.