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

Getting ready for L10n #27

Closed
wants to merge 3 commits into from
Closed

Conversation

tdd
Copy link
Contributor

@tdd tdd commented Feb 20, 2015

Hey Domenic,

First, thanks a ton for all the work you do on ES, Node, and the like. I enjoy my work and field in part because of you!

This workshop is great, too. As part of an ongoing effort in nodeschool/organizers#64 to localize all known workshops so as to reduce friction for traditionally "English-impaired" communities (e.g. FR 😞), I've been upgrading all workshopper-based workshops to the latest, g11n-capable versions of the infra, and upgrading their codebase to fit.

I also took the liberty to tweak your problem Markdown files a bit to better fit with well-established (by now) best practices, such as ~80-character wrapping, well-known headings, etc. Because this is more of a stylistic thing, I put that in a separate commit, but I hope you'll agree with the intent.

I'm trying to get the FR L10n for this in today (it would be great to have the npm module ready for tomorrow's first NodeSchool Paris event), but I must finish translating a fork of stream-adventure first, so it might not get in before Monday… We'll see.

I am, of course, available for any discussion on this.

Best,

@tdd tdd mentioned this pull request Feb 20, 2015
@domenic
Copy link
Owner

domenic commented Feb 26, 2015

Superceded by #28.

@domenic domenic closed this Feb 26, 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