Skip to content

added Dutch numbers to words support#166

Merged
MehdiK merged 7 commits into
Humanizr:masterfrom
ryanheath:DutchNumberToWords
Apr 12, 2014
Merged

added Dutch numbers to words support#166
MehdiK merged 7 commits into
Humanizr:masterfrom
ryanheath:DutchNumberToWords

Conversation

@ryanheath

Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

You have some weird spacing issues throughout the code! Please reformat your code.

@MehdiK

MehdiK commented Apr 10, 2014

Copy link
Copy Markdown
Collaborator

There are 10 failing unit tests. Please fix these. Also please add your PR to the release-notes file.

@ryanheath

Copy link
Copy Markdown
Contributor Author

Wow, how did that happen?!

I'll fix it shortly!

// Ryan

On Thu, Apr 10, 2014 at 4:50 PM, Mehdi Khalili notifications@github.comwrote:

There are 10 failing unit tests. Please fix these. Also please add your PR
to the release-notes file.

Reply to this email directly or view it on GitHubhttps://github.com//pull/166#issuecomment-40092375
.

@MehdiK

MehdiK commented Apr 12, 2014

Copy link
Copy Markdown
Collaborator

Sorry. GitHub doesn't send notifications for new commits and your changes slipped under my radar. Now the whole thing has changed so much it's quite hard to rebase your changes o_O

I think the easiest way would be for you to pull from upstream and apply your changes again as rebase is going to be quite painful. Sorry

@ryanheath

Copy link
Copy Markdown
Contributor Author

I have rebased my changes on the latest version.

// Ryan

On Sat, Apr 12, 2014 at 3:29 PM, Mehdi Khalili notifications@github.comwrote:

Sorry. GitHub doesn't send notifications for new commits and your changes
slipped under my radar. Now the whole thing has changed so much it's quite
hard to rebase your changes o_O

I think the easiest way would be for you to pull from upstream and apply
your changes again as rebase is going to be quite painful. Sorry

Reply to this email directly or view it on GitHubhttps://github.com//pull/166#issuecomment-40280355
.

@MehdiK MehdiK merged commit 5e2ffe7 into Humanizr:master Apr 12, 2014
@MehdiK

MehdiK commented Apr 12, 2014

Copy link
Copy Markdown
Collaborator

Splendid! Thanks a lot.

@MehdiK

MehdiK commented Apr 14, 2014

Copy link
Copy Markdown
Collaborator

This is now on NuGet as v1.22.1. Thanks for the contribution.

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