-
Notifications
You must be signed in to change notification settings - Fork 476
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
added swedish language including test cases #352
Conversation
Pull Request Test Coverage Report for Build 974
💛 - Coveralls |
@peternordstrom could you please, add tests for the missing lines on https://coveralls.io/builds/35506839/source?filename=num2words%2Flang_SV.py#L98? We can't merge the PR if the coverage decreases. |
…r options not implemented.
@mromdhane, I have updated the code so that it passes all tests now. See if it looks ok to you. |
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.
@peternordstrom thanks for updating the tests. I forgot to suggest you update the README.md also to include Swedish on the list of languages, could you do it, please.
Any intention to implement year
, currency
and ordinal
?
I updated the Readme now.
I might implement year, currency and numordinal later but for now, this is
what I have time for. And I believe these are the most important
implementations.
Cheers
Peter
…On Thu, 28 Jan 2021, 05:19 Ernesto Rodriguez Ortiz, < ***@***.***> wrote:
***@***.**** commented on this pull request.
@peternordstrom <https://github.com/peternordstrom> thanks for updating
the tests. I forgot to suggest you update the README.md also to include
Swedish on the list of languages, could you do it, please.
Any intention to implement year, currency and ordinal ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#352 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEDVLZCX46TJX2K3FXYMAD3S4DQTPANCNFSM4UQO2GUA>
.
|
@peternordstrom Thank you for adding the Swedish language and hope to continue the |
Fixes # by...
Changes proposed in this pull request:
Status
How to verify this change
Check if words look reasonable? Compare to Norweigan.
Additional notes
Swedish rules are very similar to norweigan so it was pretty easy to add this language