Closed rix0rrr closed 7 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
65de3dc
) 89.82% compared to head (f337ffd
) 90.98%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Thanks! Once released, this will be very helpful for https://github.com/hedyorg/hedy!
(Speaking of, any idea when that release might be? 😇🙏)
This upgrades the CLDR database to release 43.
I tried upgrading to 44 initially, but there seemed to be a lot of breaking changes in there that I didn't know how to deal with, and the locales I'm personally interested in are already supported in 43.
I had to make two additional changes in addition to updating the import:
One doctest, regular spaces have been replaced by thin spaces
In the importing of
parentLocales
. The newsupplementalData.xml
contains a new type of declaration:This refers to a locale called
sr_ME
, but there is no.xml
file for it and so no.dat
file gets generated either. At runtime, when the locale forsr_Cyrl_ME
is looked up, we try to merge the data fromsr_ME
into it and then get aFileNotFound
exception whensr_ME.dat
doesn't exist.From the description of this new feature:
I figured that since this type of information is new and CLDR-42 didn't have it yet, it wouldn't hurt to just ignore it for now. We don't get the benefit of the new inheritable information, but we don't break either and at least we'll be able to consume the new core data for new locales.
This change adds support for the following new locales: