Skip to content

feat: Update translations for Lao locale#37

Merged
lzwind merged 3 commits intolinuxdeepin:masterfrom
JWWTSL:master
Aug 7, 2025
Merged

feat: Update translations for Lao locale#37
lzwind merged 3 commits intolinuxdeepin:masterfrom
JWWTSL:master

Conversation

@JWWTSL
Copy link
Contributor

@JWWTSL JWWTSL commented Aug 7, 2025

No description provided.

@github-actions
Copy link

github-actions bot commented Aug 7, 2025

TAG Bot

TAG: 6.0.10
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见如下:

  1. 版本更新日志

    • debian/changelog文件中,版本从6.0.9更新到6.0.10,但未提供具体的更新内容。建议在更新日志中添加详细说明,以便其他开发者了解本次更新的具体改动。
  2. 翻译文件格式

    • translations/deepin-lianliankan_lo.tstranslations/desktop/desktop_lo.ts文件中,<location>标签的filename属性值中包含了文件路径,但格式不正确。正确的格式应该是相对于项目根目录的相对路径,例如../src/source/closewindowdialog.cpp
  3. 翻译内容

    • translations/deepin-lianliankan_lo.ts文件中,<source>标签的文本与<translation>标签的文本之间存在不一致。例如,<source> Play Again </source>的翻译是ຫຼິ້ນອີກຄັ້ງ,而<source> Have a Rest </source>的翻译是ພັກຜ່ອນ。建议确保<source><translation>标签的文本内容一致。
  4. 文件编码

    • translations/deepin-lianliankan_lo.tstranslations/desktop/desktop_lo.ts文件中,<?xml version="1.0" encoding="utf-8"?>声明中,encoding属性值应为"UTF-8",而不是"utf-8"。建议统一使用大写字母。
  5. 文件命名

    • translations/desktop/desktop_lo.ts文件中,<location>标签的filename属性值中包含了Desktop Entry]NameDesktop Entry]GenericName,这可能是拼写错误。正确的文件名应该是Desktop Entry.desktop
  6. 代码注释

    • translations/deepin-lianliankan_lo.tstranslations/desktop/desktop_lo.ts文件中,缺少对翻译文件结构的注释说明。建议添加注释,以便其他开发者了解文件的结构和用途。

综上所述,建议对上述问题进行修改和优化,以提高代码质量和可维护性。

@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

@lzwind lzwind merged commit f4ea44f into linuxdeepin:master Aug 7, 2025
16 of 19 checks passed
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