Skip to content

fix: AI translations update#94

Merged
lzwind merged 1 commit intolinuxdeepin:masterfrom
JWWTSL:master
Jul 1, 2025
Merged

fix: AI translations update#94
lzwind merged 1 commit intolinuxdeepin:masterfrom
JWWTSL:master

Conversation

@JWWTSL
Copy link
Contributor

@JWWTSL JWWTSL commented Jul 1, 2025

log: update translations file.

Please do not send pull requests to the linuxdeepin/*

see https://github.com/linuxdeepin/developer-center/wiki/Contribution-Guidelines-for-Developers

Thanks!

Summary by Sourcery

Update and refine localization strings in translation (.ts) files across multiple languages. Corrections include proper escaping of quotes, adjusted source line references, and enhanced wording for disk error messages, UI labels, and button texts to improve clarity and consistency.

Enhancements:

  • Escape double quotes in "Auto scan USB storage devices" across translations.
  • Adjust location line numbers for translation entries to match source code changes.
  • Revise disk error, mounting, and decompression messages for improved clarity in various locales.
  • Standardize key UI strings (e.g. Boot Maker description, "OK", "Cancel") and remove redundant wording in translations.

@sourcery-ai
Copy link

sourcery-ai bot commented Jul 1, 2025

Reviewer's Guide

This pull request updates translation files by standardizing source strings (escaping quotes), bumping location line numbers, normalizing TS headers, and revising various locale translations for accuracy and completeness.

File-Level Changes

Change Details Files
Standardize source string quoting
  • Replaced literal double quotes in <source> elements with &quot;
  • Unified Disk Format Error source lines across contexts
src/translations/deepin-boot-maker_pam.ts
src/translations/deepin-boot-maker_ky@Arab.ts
src/translations/deepin-boot-maker_ast.ts
src/translations/deepin-boot-maker_my.ts
src/translations/deepin-boot-maker_bqi.ts
src/translations/deepin-boot-maker_mr.ts
src/translations/deepin-boot-maker_sc.ts
src/translations/deepin-boot-maker_lo.ts
src/translations/deepin-boot-maker_sw.ts
src/translations/deepin-boot-maker_ku_IQ.ts
src/translations/deepin-boot-maker_eu.ts
src/translations/deepin-boot-maker_kn_IN.ts
src/translations/deepin-boot-maker_ta.ts
src/translations/deepin-boot-maker_hy.ts
src/translations/deepin-boot-maker_ne.ts
src/translations/deepin-boot-maker_ku.ts
src/translations/deepin-boot-maker_fil.ts
src/translations/deepin-boot-maker_ml.ts
src/translations/deepin-boot-maker_mn.ts
src/translations/deepin-boot-maker_eo.ts
src/translations/deepin-boot-maker_af.ts
src/translations/deepin-boot-maker_tzm.ts
src/translations/deepin-boot-maker_br.ts
src/translations/deepin-boot-maker_sv.ts
src/translations/deepin-boot-maker_bn.ts
src/translations/deepin-boot-maker_id.ts
Bump translation location line numbers
  • Updated line attributes in <location> tags for BMHandler, BMWindow and view contexts
  • Shifted all C++ reference line numbers by a few lines
src/translations/deepin-boot-maker_pam.ts
src/translations/deepin-boot-maker_ky@Arab.ts
src/translations/deepin-boot-maker_ast.ts
src/translations/deepin-boot-maker_my.ts
src/translations/deepin-boot-maker_bqi.ts
src/translations/deepin-boot-maker_mr.ts
src/translations/deepin-boot-maker_sc.ts
src/translations/deepin-boot-maker_lo.ts
src/translations/deepin-boot-maker_sw.ts
src/translations/deepin-boot-maker_ku_IQ.ts
src/translations/deepin-boot-maker_eu.ts
src/translations/deepin-boot-maker_kn_IN.ts
src/translations/deepin-boot-maker_ta.ts
src/translations/deepin-boot-maker_hy.ts
src/translations/deepin-boot-maker_ne.ts
src/translations/deepin-boot-maker_ku.ts
src/translations/deepin-boot-maker_fil.ts
src/translations/deepin-boot-maker_ml.ts
src/translations/deepin-boot-maker_mn.ts
src/translations/deepin-boot-maker_eo.ts
src/translations/deepin-boot-maker_af.ts
src/translations/deepin-boot-maker_tzm.ts
src/translations/deepin-boot-maker_br.ts
src/translations/deepin-boot-maker_sv.ts
src/translations/deepin-boot-maker_bn.ts
src/translations/deepin-boot-maker_id.ts
Normalize XML headers in AST translation
  • Added encoding="utf-8" to XML declaration
  • Inserted DOCTYPE on its own line for conformity
src/translations/deepin-boot-maker_ast.ts
Revise and improve locale translations
  • Overhauled Tagalog (pam), Indonesian (id), Filipino (fil) and others
  • Removed type=\"unfinished\" where translations were supplied
  • Refined wording for various error and button texts
src/translations/deepin-boot-maker_pam.ts
src/translations/deepin-boot-maker_fil.ts
src/translations/deepin-boot-maker_id.ts
src/translations/deepin-boot-maker_my.ts
src/translations/deepin-boot-maker_bn.ts
src/translations/deepin-boot-maker_eo.ts
src/translations/deepin-boot-maker_sw.ts

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey @JWWTSL - I've reviewed your changes - here's some feedback:

  • Several translation entries are still marked as type="unfinished" despite having content; please remove the unfinished attribute for those so they’re included at runtime.
  • TS file headers were changed inconsistently; ensure every .ts starts with <!–?xml version="1.0" encoding="utf-8"?–> and a matching line for proper parsing.
  • Please run the Qt lrelease (or your localization pipeline) to confirm all updated TS files compile without errors and that no source strings are missing or mismatched.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Several translation entries are still marked as type="unfinished" despite having content; please remove the unfinished attribute for those so they’re included at runtime.
- TS file headers were changed inconsistently; ensure every .ts starts with <!–?xml version="1.0" encoding="utf-8"?–> and a matching <!DOCTYPE TS> line for proper parsing.
- Please run the Qt lrelease (or your localization pipeline) to confirm all updated TS files compile without errors and that no source strings are missing or mismatched.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

log: process translations.
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查结果如下:

  1. 版本更新日志

    • debian/changelog文件中,版本从6.0.8更新到6.0.9,但未提供更新内容或更改的详细信息。建议添加具体的更改说明,以便其他开发者或用户了解更新的内容。
  2. 翻译文件删除

    • deepin-boot-maker_en_AU.tsdeepin-boot-maker_ky@Arab.ts文件被删除,但未提供替代方案或说明。如果这些文件不再需要,应确保相关功能在其他语言文件中得到了适当的翻译。
  3. 未完成的翻译

    • deepin-boot-maker_ky@Arab.ts文件中存在未完成的翻译(<translation type="unfinished"></translation>),这可能会导致用户界面显示不完整或错误。建议完成这些翻译,或者标记为待处理。
  4. 代码注释和文档

    • 检查是否有必要在代码中添加注释或文档,以提高代码的可读性和可维护性。特别是对于复杂的逻辑或重要的功能,应该添加详细的注释。
  5. 错误处理

    • 确保代码中正确处理了所有可能的错误情况,包括文件操作、网络请求等。对于可能出现的异常,应该有相应的错误处理机制。
  6. 代码风格和一致性

    • 检查代码风格是否一致,包括缩进、命名约定等。确保代码遵循项目的编码规范。
  7. 性能优化

    • 对于性能敏感的部分,检查是否有优化的空间。例如,避免在循环中进行不必要的计算或文件操作。
  8. 安全性

    • 确保代码中处理用户输入时,进行了适当的验证和清理,以防止安全漏洞,如SQL注入、XSS攻击等。
  9. 测试

    • 对于新添加或修改的功能,应该编写相应的单元测试和集成测试,以确保代码的正确性和稳定性。
  10. 依赖管理

    • 检查是否有必要更新或移除不再使用的依赖项,以保持项目的依赖关系清晰和最新。

以上是针对提交的代码审查意见,具体还需要根据项目的具体情况进行调整。

@github-actions
Copy link

github-actions bot commented Jul 1, 2025

TAG Bot

TAG: 6.0.9
EXISTED: no
DISTRIBUTION: unstable

@lzwind lzwind merged commit b595a12 into linuxdeepin:master Jul 1, 2025
16 of 19 checks passed
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: JWWTSL, lzwind

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants