-
Notifications
You must be signed in to change notification settings - Fork 88
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
Dutch language integration #69
Conversation
Hi @MaciejLipinski, could you kindly take a look at this pull request? I'm available for discussions. |
Hi, any news on this? |
Hey, is this project still supported? |
Hi @GHim9182, sorry for no answer - temporarily there was no person taking care of this project. |
src/test/groovy/pl/allegro/finance/tradukisto/internal/languages/dutch/DutchValuesTest.groovy
Show resolved
Hide resolved
src/test/groovy/pl/allegro/finance/tradukisto/internal/languages/dutch/DutchValuesTest.groovy
Show resolved
Hide resolved
src/test/groovy/pl/allegro/finance/tradukisto/internal/languages/dutch/DutchValuesTest.groovy
Show resolved
Hide resolved
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.
Before I will check whole code, we need to add more test cases in unit tests.
# Conflicts: # src/main/java/pl/allegro/finance/tradukisto/ValueConverters.java
Hi @jglaszka, it's done. I also merged all your changes of the last few days in the code. Could you check? |
I will take a look at this today. |
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.
Looks good to me. :) tommorow I will merge this
No description provided.