-
Notifications
You must be signed in to change notification settings - Fork 183
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.
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
Move icu_num_utils to utils/num #303
Conversation
An open question is whether we want to have |
Notice: the branch changed across the force-push!
~ Your Friendly Jira-GitHub PR Checker Bot |
+1 for The package name should be either "icu-fixed-decimal" or "fixed-decimal", whichever makes you happier. |
great! I named it |
Notice: the branch changed across the force-push!
~ Your Friendly Jira-GitHub PR Checker Bot |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Fixes #167.
This just moves the
icu_num_utils
component to beutils/num
and renames it toicu_utils_num
.I'm open to discuss whether the latter is necessary. My motivation is that it allows us to collect all utils as
icu_utils
andnum_utils
is just one of them.