-
-
Notifications
You must be signed in to change notification settings - Fork 188
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
-Issue#212: Adding more machine translation alternatives #247
base: master
Are you sure you want to change the base?
Conversation
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.
Thanks for the PR. I added some comments.
Also can you please add the new translators on the README
Added the translations to README. |
@rupeshkumaar I approved your PR but The tests are failing. Can you check it out or maybe rebase/merge your branch with the latest of master |
@nidhaloff the branch is up to date with the master branch. So, it does not seem to do anything with that. It said poetry installation failed in one of the tests. Can you please check that? |
Please review now and merge @nidhaloff |
I have worked on #212: Adding more machine translation alternatives (Glosbe & Wikimedia MinT)
Please review and merge: @nidhaloff