Skip to content

chore: update version to 6.5.55#268

Merged
lzwind merged 1 commit intolinuxdeepin:masterfrom
pengfeixx:master
Apr 29, 2026
Merged

chore: update version to 6.5.55#268
lzwind merged 1 commit intolinuxdeepin:masterfrom
pengfeixx:master

Conversation

@pengfeixx
Copy link
Copy Markdown
Contributor

  • bump version to 6.5.55

Log : bump version to 6.5.55

- bump version to 6.5.55

Log : bump version to 6.5.55
@github-actions
Copy link
Copy Markdown

TAG Bot

TAG: 6.5.55
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot
Copy link
Copy Markdown

deepin pr auto review

这段代码 diff 包含了 debian/changeloglinglong.yaml 两个文件的修改,主要涉及版本号的更新和变更日志的记录。以下是对这两处修改的详细审查和改进意见:

1. 语法逻辑审查

  • 版本号一致性 (严重)

    • debian/changelog 中,新条目的版本号声明为 6.5.55
    • linglong.yaml 中,版本号更新为 6.5.55.1
    • 问题:这两个文件中的版本号不一致。通常 debian/changelog 的版本号应与构建配置文件(如 linglong.yaml)保持一致,或者遵循特定的构建规则(例如构建系统自动追加 .1)。如果不一致,会导致构建出的包版本混乱,难以追踪。
    • 建议:请确认项目的版本管理规范。如果 linglong.yaml 必须带有 .1 的后缀,那么 debian/changelog 通常也需要保持一致,或者在构建流程中明确说明两者的映射关系。作为通用建议,两者应保持为 6.5.55
  • 变更日志内部逻辑 (严重)

    • debian/changelog 中写道:chore: update version to 6.5.54
    • 问题:该条目的头部声明的是版本 6.5.55,但日志内容却写着更新到 6.5.54。这是一个明显的逻辑错误。如果是发布 6.5.55,日志内容不应包含"更新到旧版本"的描述,除非这是一个回滚操作(但根据 feat 描述并非如此)。
    • 建议:删除 chore: update version to 6.5.54 这一行,或者将其修正为 chore: update version to 6.5.55(虽然通常版本号变更由头部隐含,不需要单独写一行)。
  • 时间戳 (次要)

    • debian/changelog 中的日期是 Wed, 29 Apr 2026
    • 问题:年份是 2026 年,这可能是系统时间设置错误或者是笔误。
    • 建议:修正为当前年份(例如 2023 或 2024)。

2. 代码质量与规范

  • Commit Message 规范

    • 使用了 featchore 前缀,符合 Conventional Commits 规范,这一点做得很好。
    • feat(shortcuts): add arrow key scrolling support for document viewer 描述清晰,指明了影响范围和具体功能。
  • Debian Changelog 格式

    • 格式基本符合 Debian 标准,但如前所述,内容逻辑有误。

3. 代码性能

  • N/A:本次修改仅涉及元数据和配置文件,不涉及代码逻辑,因此对性能无直接影响。

4. 代码安全

  • N/A:不涉及安全漏洞。

修改建议总结

建议将 debian/changelog 修改为以下内容,以修复逻辑错误和版本号不一致的问题:

diff --git a/debian/changelog b/debian/changelog
index 4917100d..35c6c882 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+deepin-reader (6.5.55) unstable; urgency=medium
+
+  * feat(shortcuts): add arrow key scrolling support for document viewer
+
+ -- xiepengfei <xiepengfei@uniontech.com>  Wed, 29 Apr 2024 16:50:58 +0800
+
 deepin-reader (6.5.54) unstable; urgency=medium
 
   * [skip CI] Translate deepin-reader.ts in pt_BR

同时,请确认 linglong.yaml 中的版本号 6.5.55.1 是否正确。如果项目要求 linglong.yaml 必须比 debian/changelog 多一位构建号,请保持现状;否则建议统一为 6.5.55

@deepin-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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 0f2c364 into linuxdeepin:master Apr 29, 2026
11 checks passed
@deepin-bot
Copy link
Copy Markdown
Contributor

deepin-bot Bot commented Apr 29, 2026

TAG Bot

Tag created successfully

📋 Tag Details
  • Tag Name: 6.5.55
  • Tag SHA: 9882149c51565750a04408bc77a316f66acc4910
  • Commit SHA: 1641c86e666523e7aea2031ee86a74a1da81b9ea
  • Tag Message:
    Release deepin-reader 6.5.55
    
    
  • Tagger:
    • Name: pengfeixx
  • Distribution: unstable

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