Skip to content

chore: update version to 6.5.51#619

Merged
lzwind merged 1 commit into
linuxdeepin:masterfrom
wyu71:master
May 19, 2026
Merged

chore: update version to 6.5.51#619
lzwind merged 1 commit into
linuxdeepin:masterfrom
wyu71:master

Conversation

@wyu71

@wyu71 wyu71 commented May 19, 2026

Copy link
Copy Markdown
Contributor
  • bump version to 6.5.51

Log : bump version to 6.5.51

- bump version to 6.5.51

Log : bump version to 6.5.51
@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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 4767e21 into linuxdeepin:master May 19, 2026
15 of 19 checks passed
@deepin-ci-robot

Copy link
Copy Markdown

deepin pr auto review

你好!我是CodeGeeX。我已经仔细审查了你提供的 Git Diff 输入。本次修改主要涉及版本号的升级以及更新日志的记录。

从代码规范、逻辑和安全性的角度来看,本次修改虽然简单,但存在一些逻辑不一致规范性问题。以下是详细的审查意见:

1. 语法逻辑

  • 版本号不一致
    • debian/changelog 中,新增的版本号被标记为 6.5.51,但在描述中却写着 * chore: update version to 6.5.50
    • linglong.yaml 中,版本号从 6.5.50.1 更新到了 6.5.51.1
    • 问题:Changelog 的描述与实际变更的版本号不符。既然是升级到 6.5.51,描述应为 update version to 6.5.51
  • 版本号命名规则不一致
    • debian/changelog 中的版本号是 6.5.51(三段式)。
    • linglong.yaml 中的版本号是 6.5.51.1(四段式)。
    • 问题:虽然不同打包系统允许有不同的版本号规范,但通常主版本号应保持同步。如果 .1 是 linglong 特有的迭代号,则无妨;但如果是疏忽漏改,则会导致版本管理混乱。建议确认这两处的版本号策略是否预期一致。

2. 代码质量

  • Commit Message 与 Changelog 描述不匹配
    • Changelog 中的修复项是 fix(scrollbar): replace JS autoHide with CSS-based auto-hide to fix cursor flicker,这是一项代码修复。
    • 紧接着的描述是 chore: update version to 6.5.50
    • 问题:按照常规的 Git 提交和 Changelog 规范,chore: update version 通常是指纯粹的发版提交,不应与 fix 混在同一个版本记录中,或者如果混在一起,chore 的描述必须准确反映当前的版本号。
  • Changelog 日期格式
    • 日期为 Tue, 19 May 2026 21:30:11 +0800
    • 问题:2026年是未来的时间。虽然 Debian 的 debchange 工具有时会因为系统时间错误生成未来时间,但这会导致打包时出现警告(debian-changelog-file-has-wrong-day 或时间顺序问题)。请确认这是否是系统时间设置错误导致的。

3. 代码性能

  • 本次修改仅涉及配置和文档文件,不涉及核心逻辑代码,因此无性能影响
  • 附带说明:Changelog 中提到的 replace JS autoHide with CSS-based auto-hide 是一个非常好的性能优化方向。CSS 动画/过渡由浏览器渲染引擎直接处理,通常能触发 GPU 加速,且避免了 JS 频繁操作 DOM 引起的重排/重绘,对性能和流畅度都有显著提升。

4. 代码安全

  • 本次修改无安全风险

💡 改进建议

  1. 修正 Changelog 描述:将 update version to 6.5.50 改为 update version to 6.5.51
  2. 检查系统时间:确认生成 Changelog 时的系统时间是否正确,如果是误操作,请修正日期。
  3. 统一版本号认知:确认 linglong.yaml 中的 6.5.51.1 结尾的 .1 是否是必须的,如果不需要,建议与 Changelog 保持严格一致改为 6.5.51;如果是 linglong 规范,则保留,但需在团队内明确。

✅ 修正后的预期 Diff

diff --git a/debian/changelog b/debian/changelog
index 8e6dd011..8dd177d2 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+deepin-manual (6.5.51) unstable; urgency=medium
+
+  * fix(scrollbar): replace JS autoHide with CSS-based auto-hide to fix cursor flicker
+  * chore: update version to 6.5.51
+
+ -- Wang Yu <wangyu@uniontech.com>  Tue, 19 May 2024 21:30:11 +0800  /* 假设年份应为2024,请根据实际情况修改 */
+
 deepin-manual (6.5.50) unstable; urgency=medium
 
   * fix(scrollbar): prevent horizontal scrollbar and fix bottom content clipping
diff --git a/linglong.yaml b/linglong.yaml
index 43bf871c..cb1df9be 100755
--- a/linglong.yaml
+++ b/linglong.yaml
@@ -7,7 +7,7 @@ version: "1"
 package:
   id: org.deepin.manual
   name: "deepin-manual"
-  version: 6.5.50.1
+  version: 6.5.51.1  /* 如果 linglong 规范要求四段式,则保留;否则建议改为 6.5.51 */
   kind: app
   description: |
     manual for deepin os.

@github-actions

Copy link
Copy Markdown

TAG Bot

TAG: 6.5.51
EXISTED: no
DISTRIBUTION: unstable

@deepin-bot

deepin-bot Bot commented May 19, 2026

Copy link
Copy Markdown
Contributor

TAG Bot

Tag created successfully

📋 Tag Details
  • Tag Name: 6.5.51
  • Tag SHA: c1de3f18e6c4ac0bf6d9c99269a759e60681ca8d
  • Commit SHA: b9081fdee5f0e90512267fb7d629053446e7e8d4
  • Tag Message:
    Release deepin-manual 6.5.51
    
    
  • Tagger:
    • Name: wyu71
  • 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