Skip to content

Conversation

@Kakueeen
Copy link
Contributor

update translations

Log: update translations

update translations

Log: update translations
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. 版本号更新

    • linglong.yaml文件中,版本号从6.0.25.1更新到6.0.26.1,这是一个合理的更新,但需要确保新版本包含了必要的更改日志和更新说明。
  2. 翻译文件删除

    • deepin-image-viewer_en_AU.ts文件被删除,如果这个文件不再需要,那么删除是合理的。如果这个文件仍然需要,请确保相关功能在其他翻译文件中得到了适当的处理。
  3. 翻译内容

    • deepin-image-viewer_en_AU.ts文件中,一些翻译内容被删除或未完成,例如Select picturesImage file not found。如果这些翻译内容不再需要,那么删除是合理的。如果这些内容仍然需要,请确保它们在其他翻译文件中得到了适当的处理。
  4. 代码注释和文档

    • deepin-image-viewer_en_AU.ts文件中,一些翻译内容缺少注释,这可能会影响代码的可读性和维护性。建议为每个翻译内容添加适当的注释。
  5. 代码风格

    • deepin-image-viewer_en_AU.ts文件中,一些翻译内容使用了不同的格式,这可能会导致代码风格不一致。建议统一代码风格,例如使用一致的缩进和空格。
  6. 翻译一致性

    • deepin-image-viewer_en_AU.ts文件中,一些翻译内容可能存在不一致的情况,例如FullscreenExit fullscreen的翻译。建议检查所有翻译内容,确保它们的一致性。
  7. 翻译更新

    • 如果deepin-image-viewer_en_AU.ts文件被删除,需要确保其他翻译文件中相应的翻译内容也得到了更新。

总结:

  • 确保版本号更新与实际更改相匹配。
  • 处理翻译文件的删除和更新,确保翻译内容的一致性和完整性。
  • 添加适当的注释和统一代码风格,以提高代码的可读性和维护性。

@github-actions
Copy link

TAG Bot

TAG: 6.0.26
EXISTED: no
DISTRIBUTION: unstable

@lzwind lzwind merged commit a2a6674 into linuxdeepin:master Jun 27, 2025
19 of 22 checks passed
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

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

@lzwind
Copy link
Contributor

lzwind commented Jun 27, 2025

/topic cd-20250627

@deepin-ci-robot
Copy link

Add topic: cd-20250627 successed.

@lzwind
Copy link
Contributor

lzwind commented Jun 27, 2025

/integr-topic cd-20250627

@deepin-ci-robot
Copy link

Integrated with pr deepin-community/Repository-Integration#2959

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

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants