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.

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

CLDR-17566 converting units #3833

Merged
merged 3 commits into from
Jul 11, 2024

Conversation

chpy04
Copy link
Contributor

@chpy04 chpy04 commented Jun 29, 2024

CLDR-17566

Part of the CLDR --> GFM conversion project

Converting: https://cldr.unicode.org/translation/units,
https://cldr.unicode.org/translation/units/measurement-systems,
https://cldr.unicode.org/translation/units/unit-names-and-patterns,

Text diffs can be seen in the second commit on this PR.
Use the 📄 button to see a rich view, for comparison.

ALLOW_MANY_COMMITS=true

M{0}» produces «{0}M o», but the explicit translation is «{0} Mo»
Descriptions Examples
The accent needs to shift when a prefix is added «δεκατο{0}» produces «{0} δεκατολίτρα», but the explicit translation is «{0} δεκατόλιτρα»
The internal algorithm for when to combine spacing needs adjustment. M{0}» produces «{0}M o», but the explicit translation is «{0} Mo»
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

table-like text --> md table

So the appropriate pattern for power2 would be:
3. https://st.unicode.org/cldr-apps/v#/fr/CompoundUnits/15b049cba8052719 => {0} carrés
https://st.unicode.org/cldr-apps/v#/fr/CompoundUnits/15b049cba8052719 => {0} carrés
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Converted these into actual number/bullet point lists instead of numbers and intents

@srl295 srl295 merged commit 15dd624 into unicode-org:main Jul 11, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants