Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

process.versions.icu #3089

Closed
silverwind opened this issue Sep 27, 2015 · 0 comments
Closed

process.versions.icu #3089

silverwind opened this issue Sep 27, 2015 · 0 comments
Labels
feature request Issues that request new features to be added to Node.js. i18n-api Issues and PRs related to the i18n implementation. process Issues and PRs related to the process subsystem.

Comments

@silverwind
Copy link
Contributor

This would be useful to test for the presence of ICU and to implement version-specific behaviour. I've been trying to use U_ICU_VERSION, but haven't gotten it working yet, if someone wants to take a shot.

Related: #3007
@silverwind silverwind added feature request Issues that request new features to be added to Node.js. process Issues and PRs related to the process subsystem. i18n-api Issues and PRs related to the i18n implementation. labels Sep 27, 2015
@evanlucas evanlucas mentioned this issue Sep 28, 2015
Merged
@silverwind silverwind mentioned this issue Sep 29, 2015
Closed
evanlucas added a commit to evanlucas/node that referenced this issue Sep 29, 2015
@evanlucas
src: add process.versions.icu
30b8bb0 
If i18n support is present, add the icu version
to process.versions

Fixes: nodejs#3089
PR-URL: nodejs#3102
Reviewed-By: Steven R Loomis <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
Reviewed-By: Rod Vagg <[email protected]>
evanlucas added a commit that referenced this issue Oct 8, 2015
@evanlucas @jasnell
src: add process.versions.icu
7271cb0 
If i18n support is present, add the icu version
to process.versions

Fixes: #3089
PR-URL: #3102
Reviewed-By: Steven R Loomis <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
Reviewed-By: Rod Vagg <[email protected]>
@snyk-bot snyk-bot mentioned this issue Mar 16, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request Issues that request new features to be added to Node.js. i18n-api Issues and PRs related to the i18n implementation. process Issues and PRs related to the process subsystem.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@silverwind