-
Notifications
You must be signed in to change notification settings - Fork 328
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
Add Dutch translation files #124
base: master
Are you sure you want to change the base?
Conversation
Includes a small fix in `changesoutput.py` (had an inconsistent translation key concerning historical commit information).
Thank you. The translation looks good. I'll add you to the team of translators as well :). I'll notify you when changes are needed. I'm going to include this in the 0.4.5 release. |
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.
Some minor suggestions for changes in the translation
msgstr "" | ||
"Copyright © 2012-2015 Ejwa Software. Alle rechten voorbehouden.\n" | ||
"License GPLv3+: GNU GPL versie 3 of hoger <http://gnu.org/licenses/gpl.html>.\n" | ||
"Dit programma is vrije software: je mag het herdistribueren en/of wijzigen.\n" |
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.
I would change "vrije" to "gratis" in this context
"\n" | ||
"Verplichte argumenten voor lange opties zijn ook voor korte opties verplicht.\n" | ||
"Boolean argumenten kunnen alleen voor lange opties worden gegeven.\n" | ||
" -f, --file-types=EXTENSIES een door comma's gesepareerde list van\n" |
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.
"comma's" should be "komma's"
"een van de gespecificeerde extensies veranderen, toevoegen of verwijderen,\n" | ||
"zie -f of --file-types voor meer informatie.\n" | ||
"\n" | ||
"gitinspector vereist dat de git executable in de PATH variabel ter beschikking\n" |
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.
"variabel" should be "variabele"
Fix a couple of minor mistakes pointed out by @yoshivda in the Dutch translation.
Hi @yoshivda, thanks for pointing those out! I've fixed them in the latest commit. |
Good work guys :). |
Moved to 0.5.0. |
This PR offers a Dutch translation of the gitinspector application (as referenced in #123).
Includes a small fix in
changesoutput.py
(had an inconsistent translation key concerning historical commit information).