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 #57

Merged
merged 19 commits into from
Feb 21, 2015
Merged

L10n fr #57

merged 19 commits into from
Feb 21, 2015

Conversation

tdd
Copy link
Contributor

@tdd tdd commented Feb 18, 2015

Hey again,

As mentioned in #56, this is the second step of our recent work on globalizing and localizing expressworks. This includes the commits referenced in #56, which will "disappear" when/if you merge that one first, to leave only FR-related commits for actual localization.

I remain, naturally, available for any discussion on this.

Looking forward to our getting that in,

Best,

@tdd
Copy link
Contributor Author

tdd commented Feb 20, 2015

And yay, this auto-merges again!

azat-co pushed a commit that referenced this pull request Feb 21, 2015
@azat-co azat-co merged commit ee8772c into azat-co:master Feb 21, 2015
@azat-co
Copy link
Owner

azat-co commented Feb 21, 2015

@tdd I merged both PRs and published at 2.0.0... thank you! can you double check it again? or during your nodeschool event?

@tdd
Copy link
Contributor Author

tdd commented Feb 21, 2015

Hey Azat, thanks a ton! npm does show the package.

I realize perhaps it's time to update that screenshot from the README 😉

@azat-co
Copy link
Owner

azat-co commented Mar 17, 2015

haha

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