Feat: locale generating adoption
Description
Resolve #2906 (closed)
Per upstream MR changed locale generation from Ruby to JS, thus the previous script we implemented will not be able to generate any locale file.
The current MR does two things to solve this problem
- Add our Jihu own script to generate locale files:
jh/scripts/frontend/po_to_json.js
- Add
jh/lib/tasks/gettext.rake
to override Upstream's commandgettext:compile
How to use
- Manually delete the
app.js
file$ rm app/assets/javascripts/locale/en/app.js
- Execute the command
rake gettext:compile
, expecting to regenerateapp.js
$ rake gettext:compile $ ls app/assets/javascripts/locale/en/app.js
合并请求报告
动态
changed milestone to %16.0
requested review from @orozot and @qk44077907
assigned to @jeremywu
@jeremywu,请添加类型标记:- typebug:已发布或上线的产品中所出现的缺陷。这种类型的 issue 需在 QA 验证过后才可以关闭,而下面其它类型的 issue 在相关的 merge request (MR) 合并后即可关闭。
- typefeature:对于用户可见的新功能或对现有功能的改进。
- typemaintenance:非用户可见的产品改进,如:解决之前的技术债。
或者你也可以直接添加子类型标记(例如 featureenhancement),我们会自动补全上述标记。要详细了解标记,请查阅文档:极狐 Handbook。
@jeremywu, 检测到此合并请求的状态为 Ready 且合并请求具有审核者,已为此合并请求添加标记 workflowin reviewadded workflowin review label
requested review from @luzhiyuan.deer
2 Warnings 请为此合并请求添加一个合适的子类型标记。 请为此合并请求添加一个合并请求类型。 1 Message 更新日志缺失: 若您希望为极狐Gitlab 开源版本创建更新日志,请在提交信息添加
Changelog
结尾来将其添加到更新日志。若您希望为极狐Gitlab 商业版本创建更新日志,请在上述规则的基础上额外添加
EE: true
到提交信息末尾。若此合并请求不需要更新日志,请忽略此条消息。
审核者推荐 (JH-Beta)
已检测到需要审核的改动!
请参考以下 Danger 建议的审核者和维护者:
改动分类 审核者 维护者 backend xiaofeng ( @xfyuan
)Baodong ( @icbd
)frontend Vixon ( @lxwan
)Jeremy ( @jeremywu
)代码审核步骤:
- 在合并请求准备好(非 Draft 状态)后,设置一位或多位审核者(参考上方表格的"审核者")。
- 第一次得到审核者批准时会进入安全审核流程,请等待安全审核通过。
- 在得到所有审核者批准后,设置一位项目维护者做最终审核并完成代码合并(参考上方表格的"维护者")。
更多有关代码审核的流程请参见代码审核流程。
为了更平均地分配审核请求到所有可用的审查者,Danger 会基于审核者的状态和其他信息为每个席位挑选审核者。 如果有更好的审核者人选,请放心地忽略此算法推荐的人选。
决定好审核者的人选之后,请将他们指派为审核者或者@提醒他们! Danger Bot 不会帮您做这件事。
如有需要,您可以重试生成这条评论的
danger-review
作业。Generated by
Danger由 Ghost User 已编辑 于@jeremywu LGTM
- 由 Martin Tan 已解决 于
这是此合并请求第一次被批准,现在可以开始做安全审查了。
@songhuang,你能帮忙做安全审查吗(使用"批准"功能)?
added security-reviewpending label
added 448 commits
-
b7a93c8b...11207e51 - 446 commits from branch
main-jh
- 615999a6 - Feat: locale generating adoption
- 80b0f2f8 - Remove deepmerge dependency
-
b7a93c8b...11207e51 - 446 commits from branch
added 5 commits
-
80b0f2f8...8524f331 - 3 commits from branch
main-jh
- 650b449a - Feat: locale generating adoption
- 2b6142aa - Remove deepmerge dependency
-
80b0f2f8...8524f331 - 3 commits from branch
added security-reviewapproved label and removed security-reviewpending label