chore(transifex): add configuration for power plugin translations#72
Conversation
1. Add Transifex config mapping source translation to language files 2. Set source language to en_US for plugin-power-session Log: add transifex.yaml config for power plugin translations chore(transifex): 为 power 插件添加 Transifex 翻译配置 1. 添加 Transifex 配置,映射源翻译文件到各语言文件 2. 设置源语言为 en_US 的 plugin-power-session Log: 为 power 插件添加 transifex.yaml 翻译配置
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- Consider normalizing the indentation under
settings:to match the two-space indentation used underfilters:to avoid potential YAML parsing or readability issues. - Double-check that the
translation_files_expressionpattern (plugin-power-session_<lang>.ts) aligns with your actual locale naming convention (e.g.,en_US,zh_CN) so Transifex maps locales correctly.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Consider normalizing the indentation under `settings:` to match the two-space indentation used under `filters:` to avoid potential YAML parsing or readability issues.
- Double-check that the `translation_files_expression` pattern (`plugin-power-session_<lang>.ts`) aligns with your actual locale naming convention (e.g., `en_US`, `zh_CN`) so Transifex maps locales correctly.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
deepin pr auto review你好!我是CodeGeeX。我已仔细审查了你提供的 总体来说,该配置结构清晰,基本满足了将 Qt 翻译源文件与 Transifex 关联的需求。但在语法逻辑、代码质量、代码性能和代码安全方面,我有一些改进建议: 1. 语法逻辑
2. 代码质量
3. 代码性能
4. 代码安全
改进后的代码建议filters:
- filter_type: file
name: power-session-translations # 改进:增加具有业务语义的名称,便于管理
source_file: src/plugin-qt/power/session/translations/plugin-power-session_en.ts
file_format: QT
source_language: en_US
translation_files_expression: src/plugin-qt/power/session/translations/plugin-power-session_<lang>.ts
settings:
pr_branch_name: transifex_update_<br_unique_id>详细解释:
如果你在后续集成 CI/CD(如 GitHub Actions 或 GitLab CI)来调用这个配置文件时遇到问题,欢迎随时向我提问! |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideAdds a Transifex configuration for the power session plugin, defining how the English source Qt translation file maps to per-language translation files and how Transifex-generated PR branches are named. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: BLumia, mhduiy The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/forcemerge |
|
This pr force merged! (status: blocked) |
Log: add transifex.yaml config for power plugin translations
chore(transifex): 为 power 插件添加 Transifex 翻译配置
Log: 为 power 插件添加 transifex.yaml 翻译配置
Summary by Sourcery
Build: