Skip to content

chore: bump version to 1.3.49#264

Merged
deepin-bot[bot] merged 1 commit intolinuxdeepin:masterfrom
Johnson-zs:master
Mar 5, 2026
Merged

chore: bump version to 1.3.49#264
deepin-bot[bot] merged 1 commit intolinuxdeepin:masterfrom
Johnson-zs:master

Conversation

@Johnson-zs
Copy link
Contributor

1.3.49

Log:

@github-actions
Copy link

github-actions bot commented Mar 5, 2026

TAG Bot

TAG: 1.3.49
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Johnson-zs

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

@deepin-ci-robot
Copy link

deepin pr auto review

这份 debian/changelog 的变更记录整体上符合 Debian 的 changelog 格式规范,但在一些细节和逻辑上存在可以改进的地方。以下是我的审查意见:

1. 语法与格式

  • 日期逻辑错误

    • 问题:版本号是 1.3.49,但维护者日期写的是 2026 年(Thu, 05 Mar 2026)。除非这是为了未来的特定里程碑预留的条目,否则这看起来像是一个笔误。
    • 建议:请核实年份。如果是当前开发,应改为当前年份(如 2024 或 2025)。
  • 提交信息大小写不一致

    • 问题:使用了 chore:feat:Fix:。第三项 Fix: 的首字母是大写,而前两项是小写。
    • 建议:保持风格统一。通常在 Debian changelog 中,首字母大写是常见的做法,或者如果遵循 Conventional Commits 规范,通常使用小写。建议统一为小写(fix:)或统一为首字母大写(Fix:)。

2. 代码质量与逻辑

  • 变更描述的清晰度

    • chore: update copyright years to 2026:同上,除非是预知未来,否则年份 2026 与当前时间不符。
    • Fix: [file-enumerator] use g_file_enumerator_next_file()
      • 问题:这看起来像是一个技术实现的细节,但没有说明为什么要做这个修改。是为了修复崩溃?还是为了提升性能?或者是修复内存泄漏?
      • 建议:补充修复的具体原因。例如:Fix: [file-enumerator] use g_file_enumerator_next_file() to prevent blocking UI thread(防止阻塞 UI 线程)或者 Fix: [file-enumerator] memory leak by using ...(修复内存泄漏)。
  • 功能变更的范围

    • feat: add filename blacklist filtering for search index:这是一个新功能。通常 feat 类型的变更在 unstable 发布中是可以接受的,但需确保这个功能不会意外地过滤掉用户需要的文件。

3. 性能

  • 关于 g_file_enumerator_next_file() 的变更
    • 审查点:虽然 diff 中只有描述,但 g_file_enumerator_next_file() 是同步调用。如果在主线程中大量使用此函数,可能会导致界面卡顿。
    • 建议:请确认代码实现中是否正确处理了异步操作,或者是否在后台线程中调用了该函数,以确保性能不受影响。

4. 安全

  • 版权年份
    • 审查点:虽然不是代码安全问题,但错误的版权年份(如 2026)可能会在法律合规性审查或开源许可证合规性检查中引起不必要的麻烦。
    • 建议:修正为正确的年份。

改进后的建议示例

假设当前年份是 2024 年,且修复 g_file_enumerator_next_file 是为了解决性能阻塞问题,建议修改如下:

 util-dfm (1.3.49) unstable; urgency=medium
 
-  * chore: update copyright years to 2026
-  * feat: add filename blacklist filtering for search index
-  * Fix: [file-enumerator] use g_file_enumerator_next_file()
+  * chore: update copyright years to 2024.
+  * feat: add filename blacklist filtering for search index.
+  * fix: [file-enumerator] use g_file_enumerator_next_file() to avoid UI freezing.
 
- -- Zhang Sheng <zhangsheng@uniontech.com>  Thu, 05 Mar 2026 19:12:34 +0800
+ -- Zhang Sheng <zhangsheng@uniontech.com>  Thu, 05 Mar 2024 19:12:34 +0800

总结:主要问题集中在日期年份的异常以及变更日志描述的详细程度上。修正日期并统一格式,同时补充修复项的具体原因,将使这份 changelog 更加专业和可维护。

@Johnson-zs
Copy link
Contributor Author

/forcemerge

@deepin-bot
Copy link

deepin-bot bot commented Mar 5, 2026

This pr force merged! (status: blocked)

@deepin-bot deepin-bot bot merged commit fe7cb9c into linuxdeepin:master Mar 5, 2026
20 of 22 checks passed
@deepin-bot
Copy link

deepin-bot bot commented Mar 5, 2026

TAG Bot

Tag created successfully

📋 Tag Details
  • Tag Name: 1.3.49
  • Tag SHA: 2de29855c090b0aebfe7b3bb307f13d44b10d39d
  • Commit SHA: 2a020ab875b561094bf5c206f95c6a59353607b2
  • Tag Message:
    Release util-dfm 1.3.49
    
    
  • Tagger:
    • Name: Johnson-zs
  • 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.

2 participants