Skip to content

chore: update translations#387

Merged
lzwind merged 2 commits intolinuxdeepin:masterfrom
Lighto-Ku:new-master
Jun 26, 2025
Merged

chore: update translations#387
lzwind merged 2 commits intolinuxdeepin:masterfrom
Lighto-Ku:new-master

Conversation

@Lighto-Ku
Copy link
Contributor

as title

Log: update translations

as title

Log: update translations
update version

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

deepin pr auto review

代码审查意见:

  1. 翻译一致性

    • deepin-log-viewer_en_AU.tsdeepin-log-viewer_en_CA.tsdeepin-log-viewer_en_GB.ts文件中,一些翻译文本被替换为英文原文,这可能会导致用户界面显示英文而不是本地化语言。建议确认是否需要保留英文原文,或者是否应该恢复为本地化翻译。
  2. 文本格式

    • deepin-log-viewer_en_GB.ts文件中,<translation>标签内的文本存在格式问题,例如在<source>Security level for the current system: high audit only administrators can view the audit log</source>中,|-不应该存在。这可能是复制粘贴错误,应该删除。
  3. 代码注释

    • deepin-log-viewer_en_GB.ts文件中,<translation>标签内的注释应该使用<!-- 注释内容 -->格式,而不是直接在文本中添加注释。例如,<translation>Security level for the current system: high audit only administrators can view the audit log</translation>中的注释应该被移除或使用注释标签。
  4. 文件路径和名称

    • deepin-log-viewer_en_CA.ts文件中,<translation>标签内的文本存在格式问题,例如在<translation>>Other Log</translation>中,>>应该是>。这可能是复制粘贴错误,应该修正。
  5. 代码风格

    • deepin-log-viewer_en_GB.ts文件中,<source><translation>标签内的文本应该保持一致的缩进和格式,以提高代码的可读性。
  6. 版本控制

    • debian/changelog文件中,提交信息应该简洁明了,不需要包含不必要的细节。例如,update translations.可以简化为Update translations.

综上所述,代码审查的主要关注点在于确保翻译的一致性、修正文本格式错误、保持代码注释的正确格式、修正文件路径和名称的格式问题、以及保持代码风格的统一。同时,提交信息应该简洁明了。

@github-actions
Copy link

TAG Bot

TAG: 6.5.17
EXISTED: no
DISTRIBUTION: unstable

@lzwind lzwind merged commit 2e0b3f3 into linuxdeepin:master Jun 26, 2025
16 of 19 checks passed
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Lighto-Ku, 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

@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