-
-
Notifications
You must be signed in to change notification settings - Fork 37.1k
Intl: replace icutrim.py with ICU data slicer #25136
Copy link
Copy link
Closed as not planned
Labels
i18n-apiIssues and PRs related to Node.js internationalization support.Issues and PRs related to Node.js internationalization support.stalledIssues and PRs manually marked as stalled and scheduled for automatic closure.Issues and PRs manually marked as stalled and scheduled for automatic closure.wipIssues and PRs that are still a work in progress.Issues and PRs that are still a work in progress.
Description
Activity
Metadata
Metadata
Assignees
Labels
i18n-apiIssues and PRs related to Node.js internationalization support.Issues and PRs related to Node.js internationalization support.stalledIssues and PRs manually marked as stalled and scheduled for automatic closure.Issues and PRs manually marked as stalled and scheduled for automatic closure.wipIssues and PRs that are still a work in progress.Issues and PRs that are still a work in progress.
In Node 0.12, I added icutrim.py and the config file icu_small.json to enable Node's "english only" default ICU packaging (aka
icu-small).ICU now has its own data slicer, see https://unicode-org.atlassian.net/browse/ICU-10923 (see the design doc at the top of that issue)
I'd encourage node people ( cc @nodejs/intl ) to visit the ICU issue and comment on the syntax.
Describe the solution you'd like
Less headache for Node maintainers! Less special purpose code!
Describe alternatives you've considered
do nothing - keep maintaining these python scripts