Skip to content
This repository was archived by the owner on Apr 22, 2023. It is now read-only.

New Crowdin translations #99

Merged
merged 6,670 commits into from
Jun 3, 2018
Merged

New Crowdin translations #99

merged 6,670 commits into from
Jun 3, 2018

Conversation

nodejs-crowdin-zz
Copy link
Contributor

Enjoy!

@lukaszewczak
Copy link
Contributor

Wow, this is realy big PR, what steps should be taken now, should we review it, somehow - what we shoul look for?

@JCMais
Copy link
Contributor

JCMais commented May 31, 2018

Holy, my inbox just got full. lol

How are we going to review this? 5M changes and increasing

This is also creating a really big queue on Travis

Edit: I have just enabled auto cancellation of builds for old commits on PRs at Travis

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@zeke
Copy link
Contributor

zeke commented Jun 2, 2018

The initial Crowdin sync is complete and this is ready to go. Anyone in @nodejs/i18n should feel free to approve and merge.

@zeke
Copy link
Contributor

zeke commented Jun 2, 2018

I have just enabled auto cancellation of builds for old commits on PRs at Travis

Thank you, @JCMais 👍

@lukaszewczak lukaszewczak self-requested a review June 3, 2018 05:43
lukaszewczak

This comment was marked as off-topic.

@lukaszewczak lukaszewczak merged commit c772e20 into master Jun 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants