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

Update abbreviations in german language file #590

Merged
merged 3 commits into from
Mar 17, 2021
Merged

Update abbreviations in german language file #590

merged 3 commits into from
Mar 17, 2021

Conversation

chaosflaws
Copy link
Contributor

Currently, the German language file uses the English abbreviations for "million" and so on. This is particularly unhelpful because the names for large numbers do not match at all:

  • million -> Million
  • billion -> Milliarde
  • trillion -> Billion

This exposes a bug in the unformatter's handling of byte-formatted numbers. In a string like 1Bi (for one billion), the byte unformatter finds the capital B in the string and therefore treats this as bytes, which obviously isn't intended. I fixed this by making the unformatter search for complete postfixes without additional characters.

@BenjaminVanRyseghem
Copy link
Owner

Awesome!!! Thanks a lot 😄

@BenjaminVanRyseghem
Copy link
Owner

let's wait for the CI now 😄

@BenjaminVanRyseghem BenjaminVanRyseghem merged commit 36da9e3 into BenjaminVanRyseghem:develop Mar 17, 2021
@chaosflaws
Copy link
Contributor Author

chaosflaws commented Mar 17, 2021

Oh wow! I only went to grab a coffee and wondered where my PR went... This must be the fastest merge I have seen so far 😁

@BenjaminVanRyseghem
Copy link
Owner

in numbro 2.3.3

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