-
-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
Remove All localized files of Chinese #4008
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we go forward with the merge?
CC @daniel-beck who was involved in JEP-216
Will merge it tomorrow if there is no negative feedback |
Would get confirmation from @LinuxSuRen that the PR build with the plugins installed retain all localizations removed here, and if so, proceed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fine if @LinuxSuRen confirms what @daniel-beck is asking.
Sorry about the delay. Time difference. |
@LinuxSuRen 你这个比较坑阿。一不小心升级一下,发现没本地中文翻译了阿。后来查到你这个提交才知道要安装的插件。 |
@mmh891113 您好,首先对给您的不方便表示歉意。不过,我们的这次变更是有在社区的邮件列表里讨论过的。另外,在版本发布的变更记录也有说明,同时,也在官方的微信公众号里也有公布。 |
All related plugins are published. Below are PRs:
jenkinsci/localization-zh-cn-plugin#31
jenkinsci/localization-support-plugin#1
jenkinsci/localization-zh-cn-plugin#71
Proposed changelog entries
Submitter checklist
* Use the
Internal:
prefix if the change has no user-visible impact (API, test frameworks, etc.)