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

Merge master-localization into master #15905

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

escbuild
Copy link
Collaborator

@escbuild escbuild commented Mar 6, 2025

Hi , could you please review and merge this PR?

================== Word Count Statistics begin ==================

GMT Thu Mar 6 20:42:34 2025: Localization Engineer Lead for this PR is [email protected],[email protected].

Vendor Engineer for this PR is [email protected].

Localization branch is updated.
New/Updated strings status
CHS: 5 strings (224 words) are in English
CHT: 5 strings (224 words) are in English
CSY: 5 strings (224 words) are in English
DEU: 5 strings (224 words) are in English
ESP: 5 strings (224 words) are in English
FRA: 5 strings (224 words) are in English
ITA: 5 strings (224 words) are in English
JPN: 5 strings (224 words) are in English
KOR: 5 strings (224 words) are in English
PLK: 5 strings (224 words) are in English
PTB: 5 strings (224 words) are in English
RUS: 5 strings (224 words) are in English

Please note that If the word count above shows some words in English, it is because the localization team hasn't processed all the new/updated strings yet. These words will be translated in the next translation loop.

Please merge this PR anyway as it will align the localizable file(s) content with the English file(s) content and prevent any functionality issue caused from misaligned localized file(s) versus English file(s).

If you are about to release and you are concerned about these strings showing up in English, please contact your localization contact/project team. Thanks.

=================== Word Count Statistics end ==================

@@ -1,7 +1,6 @@
<!--- Autodesk.DesignScript.Geometry.TSpline.TSplineSurface.BySphereBestFit --->
<!--- RUAGD4YGKJ6XHPIKDL7GZX63CDAFMI6KUSR6XMXEBGJJOATEI5IA --->
## BySphereBestFit – dokumentace
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ZbynekHanak-RWS Can we make sure localization CI does not remove comments in md files? FYI: @RobertGlobant20 @zeusongit

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @QilongTang, seems the comments were removed from the source en-US file. This change in doc/distrib/NodeHelpFiles/en-US/RUAGD4YGKJ6XHPIKDL7GZX63CDAFMI6KUSR6XMXEBGJJOATEI5IA.md file triggered CL align job, which opened PR #15905. Thanks.

Copy link
Contributor

@QilongTang QilongTang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One comment, other than that, LGTM

From build https://ls.jenkins.autodesk.com/job/CL/job/DynamoCore_CL/217/
Professional translation: all languages
Pseudo translation: none
@escbuild escbuild force-pushed the master-localization branch from 614b2ff to 8963ece Compare March 6, 2025 20:42
@escbuild escbuild requested a review from QilongTang March 6, 2025 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants