New Crowdin translations #3056
Merged
New Crowdin translations #3056
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
alexandrtovmach
approved these changes
Apr 1, 2020
|
I still see English-only files in this branch... |
|
Please read description:
|
|
Considering there are some files untranslated, I'll put it into "On hold" state until all of these files are translated, but if you think it's unnecessary, you can remove the label as well. |
|
@MaledongGit It's unnecessary because this PR is initial bunch of translations. I wouldn't like to wait until it become translated, because it can take long. |
|
I'm going to merge it, because it's blocking me to continue configuring localization |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Excluded /blog from translation
Updated project configuration to export only fully translated files. Related discussion with @XhmikosR : #3050
I believe that's not a good way, because we will never translate the website #3050 (comment), so it's kind of temporary solution, until I'll update build scripts.
This PR still contain some files that's not translated, but it's going from copy/pasted files between locales. Crowdin thoughts that this files is actual translation for original file, and that's why it's here. I think we can live with it for now.
@nodejs/website
@nodejs/i18n
@nodejs/i18n-api