cibernox / babel-plugin-precompile-intl

Compile translations in ICU message format to invocable functions at build time
9 stars 4 forks source link

Possible room for improvement #11

Open brahma-dev opened 3 years ago

brahma-dev commented 3 years ago

Saw some warnings that might be of interest.

warning svelte-intl-precompile > babel-plugin-precompile-intl > intl-messageformat-parser@3.6.4: We've written a new parser that's 6x faster and is backwards compatible. Please use @formatjs/icu-messageformat-parser

warning svelte-intl-precompile > babel-plugin-precompile-intl > intl-messageformat-parser > @formatjs/intl-unified-numberformat@3.3.7: We have renamed the package to @formatjs/intl-numberformat

cibernox commented 3 years ago

I’ll have a look and try to update. I hope there aren’t too many breaking changes

cibernox commented 3 years ago

It has been a long time but I tried to update and I found what sees to be a bug in @formatjs/icu-messageformat-parser that I've already reported. It seems to not be completely compatible with some apparently valid ICU strings that intl-messageformat-parser did parse properly.

In the interim I'm updating to the latest intl-messageformat-parser.