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

Support Vietnamese #152

Merged
merged 3 commits into from
Nov 29, 2017
Merged

Support Vietnamese #152

merged 3 commits into from
Nov 29, 2017

Conversation

duyetpt
Copy link
Contributor

@duyetpt duyetpt commented Nov 29, 2017

Support translate number to words for Vietnamese

@coveralls
Copy link

coveralls commented Nov 29, 2017

Coverage Status

Coverage decreased (-0.06%) to 99.232% when pulling 1f4eae3 on duyetpt:feature/vietnamese into 5998291 on kslazarev:master.

Copy link
Collaborator

@jlduran jlduran left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution!

Would you mind addressing those minor issues.

CHANGELOG.md Outdated
@@ -1,7 +1,7 @@
## 0.11.3 (Next)

### Features
* Your contribution here.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please, do not remove this line.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will revert it as default.


def hundreds(options = {})
if @figures.hundreds.to_i.zero?
translate :hundreds, 0, options
Copy link
Collaborator

Choose a reason for hiding this comment

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

Coveralls is complaining that this line is not getting tested.

Copy link
Contributor Author

@duyetpt duyetpt Nov 29, 2017

Choose a reason for hiding this comment

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

I will update test case to cover this line.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I cover it in commit ff7d3fa

@coveralls
Copy link

coveralls commented Nov 29, 2017

Coverage Status

Coverage decreased (-0.01%) to 99.277% when pulling 717b287 on duyetpt:feature/vietnamese into 5998291 on kslazarev:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 100.0% when pulling 717b287 on duyetpt:feature/vietnamese into 5998291 on kslazarev:master.

@jlduran
Copy link
Collaborator

jlduran commented Nov 29, 2017

Thank you!

@jlduran jlduran merged commit 5415d86 into kslazarev:master Nov 29, 2017
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.

3 participants