-
Notifications
You must be signed in to change notification settings - Fork 13
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
Regressions from 3.16 #45
Comments
@Awlexus, sorry for the very late reply - and thank you for the report. Clearly a poor regression which I'll work on. I am, overall, not making fast enough progress on this library. In large part because with the release of CLDR45 I released that I have not correctly understood the specification for units of measure and my implementation isn't correct. I am increasingly of the mind that I need to reimplement it from scratch (with basically the same public API). I will finalise the decision to do that - or not - in the next week. I will work on fixing this bug right away. In the current implementation, unit names can be either an atom (which indicates there is a direct localisation available) or a string (which needs to be processed to localise). This is part of the core misunderstanding on my part of the spec. |
Hi, we ran into a couple of issues updating this library from 3.16.4 to 3.17
1. Cldr.Unit.parse_unit_name returns string instead of atom
2. Cannot format Unit with unit_name "mg/dl"
The following function call does not work in 3.17 any more
The text was updated successfully, but these errors were encountered: