search

keymanapp / keyman

Keyman cross platform input methods system running on Android, iOS, Linux, macOS, Windows and mobile and desktop web
https://keyman.com/
Other
387 stars 107 forks source link

refactor(common): Consider moving from xml2js to fast-xml-parser #12208

Open mcdurdin opened 4 weeks ago

mcdurdin commented 4 weeks ago

Consider moving from xml2js to https://www.npmjs.com/package/fast-xml-parser?

Originally posted by @mcdurdin in https://github.com/keymanapp/keyman/issues/8616#issuecomment-1630013900

This arose again in discussions on kmc-convert

@srl295 we should discuss

srl295 commented 3 weeks ago

FWIW I use fast-xml-parser in https://github.com/unicode-org/cldr/tree/main/docs/charts/keyboards for those charts. But this is a simplistic use that is not attempting to validate.

Our json validation is separate from the xml validation, though, so that should not be an impact. I'm sure there will be some bumps due to differences in handling.

fast-xml-parser is at least currently maintained!