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

L10n fr #87

Merged
merged 18 commits into from
Apr 16, 2015
Merged

L10n fr #87

merged 18 commits into from
Apr 16, 2015

Conversation

tdd
Copy link
Collaborator

@tdd tdd commented Apr 16, 2015

Hey Tim!

First, gratz on this cool workshop.

As part of the nodeschool/organizers#64 g11n effort currently underway, I'm bringing all workshopper-based workshops up to snuff on the latest versions of it so they can get translated (L10n) readily.

I'm also bringing them on the latest versions of their tested libs, and re-testing all the exercises, to reduce the risk of obsolescence.

This PR both upgrades the infra and provides FR L10n with third-party proofreading already baked in. I hope you see this favorably. We're hosting regular Paris NodeSchool sessions, and it would be fantastic if this could all be merged in and bumped in npm soon 😄

I am, of course, available for all questions and discussions on this.

Best,

@timoxley
Copy link
Owner

Oh wow. Thank you.

timoxley added a commit that referenced this pull request Apr 16, 2015
@timoxley timoxley merged commit aa7812e into timoxley:master Apr 16, 2015
@timoxley
Copy link
Owner

Released as 1.0.0

@timoxley
Copy link
Owner

Thanks again! Amazing work.

@tdd
Copy link
Collaborator Author

tdd commented Apr 16, 2015

Hey Tim,

Glad you liked it! Thanks for the merge and version bump!

@timoxley
Copy link
Owner

❤️

This was referenced Apr 17, 2015
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