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

Locale French #215

Merged
merged 3 commits into from
Aug 2, 2015
Merged

Locale French #215

merged 3 commits into from
Aug 2, 2015

Conversation

LoicMahieu
Copy link
Contributor

yargs go international !

Related to #214

Added french language. Still work in progress. I would like some review from other french speaking people. I'll ask my colleagues but it'll be most better if community can review it too.

Is it a good idea to translate types ?
I wasn't confortable to translate this kind of words, closely linked with programming and computer science. In current review, it is not translated.

@nexdrew
Copy link
Member

nexdrew commented Jul 30, 2015

Awesome, thanks @LoicMahieu!

I would say the types could go either way - whatever is most familiar to French-speaking peoples. At least count could probably be translated, no?

@bcoe
Copy link
Member

bcoe commented Aug 1, 2015

@LoicMahieu I'm really happy with this, do you think it's ready to merge? we can allow others to edit after the fact.

@LoicMahieu
Copy link
Contributor Author

Review done! There were some mistakes, thankfully corrected. Types are finally translated, people I asked were comfortable with the translation.
Ready to merge!

@LoicMahieu LoicMahieu changed the title [WIP] Locale French Locale French Aug 2, 2015
@bcoe
Copy link
Member

bcoe commented Aug 2, 2015

Awesome thank you @LoicMahieu!

bcoe added a commit that referenced this pull request Aug 2, 2015
@bcoe bcoe merged commit 21a60e4 into yargs:master Aug 2, 2015
@bcoe
Copy link
Member

bcoe commented Aug 2, 2015

@LoicMahieu fr is released in yargs@3.17.1, thanks!

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