Skip to content

chore: update changelog to 6.1.87#1095

Merged
wjyrich merged 1 commit intolinuxdeepin:masterfrom
mhduiy:dev-changelog-6.1.87
Apr 24, 2026
Merged

chore: update changelog to 6.1.87#1095
wjyrich merged 1 commit intolinuxdeepin:masterfrom
mhduiy:dev-changelog-6.1.87

Conversation

@mhduiy
Copy link
Copy Markdown
Contributor

@mhduiy mhduiy commented Apr 24, 2026

更新说明

自动更新 changelog 到版本 6.1.87

变更内容

  • 更新 debian/changelog

版本信息

  • 新版本: 6.1.87
  • 目标分支: master

Summary by Sourcery

Documentation:

  • Refresh Debian changelog entries to reflect version 6.1.87.

update changelog to 6.1.87

Log: update changelog to 6.1.87
@github-actions
Copy link
Copy Markdown

TAG Bot

TAG: 6.1.87
EXISTED: no
DISTRIBUTION: unstable

@sourcery-ai
Copy link
Copy Markdown

sourcery-ai Bot commented Apr 24, 2026

Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

Updates the Debian package changelog to reflect the new release version 6.1.87 targeting master.

File-Level Changes

Change Details Files
Update Debian changelog metadata for new release version 6.1.87.
  • Append or modify the top changelog entry to set the version to 6.1.87.
  • Ensure the changelog entry matches the expected Debian format and references the correct target branch (master).
debian/changelog

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

Copy link
Copy Markdown

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@deepin-ci-robot
Copy link
Copy Markdown

deepin pr auto review

这份 debian/changelog 的 diff 记录了版本 6.1.87 的更新内容。从语法和格式上看,它遵循了 Debian 的 changelog 规范,没有明显的语法错误。

以下是对该变更的详细审查意见,分为格式规范、逻辑内容、代码质量及潜在风险四个方面:

1. 格式规范

  • 符合标准:该 diff 严格遵循了 Debian changelog 的格式要求(package (version) distribution; urgency),作者信息和日期格式也是标准的。
  • 格式正确:缩进(两个空格)和条目列表格式(* 开头)均正确无误。

2. 逻辑内容

  • 版本号:版本号从 6.1.86 升级至 6.1.87,符合正常的版本迭代逻辑。
  • 修改项分析
    • remove wallpaper blur and effect features:移除了壁纸模糊和特效功能。
    • migrate ImageBlur1 DBus call:迁移了 ImageBlur1 的 DBus 调用。
    • 逻辑一致性:这两点放在一起是合乎逻辑的。通常迁移底层调用(如从本地实现迁移到 DBus 服务)是重构的第一步,随后可能会移除旧的本地实现代码。
    • 依赖清理cleanup systemd service dependencies 表明对 systemd 服务依赖进行了清理,这通常与性能优化或解决启动循环有关。

3. 代码质量

  • 描述清晰度
    • 第一条变更中,"migrate ImageBlur1 DBus call" 稍微有些简略。建议补充说明是迁移到哪里(例如:从内部实现迁移到 dde-api 服务)。
    • 建议修改为:migrate ImageBlur1 DBus call to [TargetService/Module],这样后续维护者能更清楚地知道迁移的目标。
  • 分类标签:使用了 perf(性能)标签。虽然移除功能确实可能带来性能提升,但如果这次变更的主要目的是重构或功能转移,使用 refactor(重构)或 chore(构建/工具)标签可能更准确。不过,如果目的是为了减少资源占用,perf 也是可接受的。

4. 潜在风险与安全

  • 破坏性变更remove wallpaper blur and effect features 是一个破坏性变更。
    • 审查意见:需要确认这些功能是否完全由新的 DBus 调用替代了。如果新的 DBus 服务不可用,或者调用失败,用户体验将直接缺失这些功能,且没有回退机制。
    • 建议:确保在代码层面有对新 DBus 服务可用性的检查,或者在上游依赖中明确声明了对新服务的依赖。
  • 日期异常
    • diff 中的日期是 Fri, 24 Apr 2026。这是一个未来的日期。
    • 审查意见:这很可能是系统时间错误或者笔误。虽然这不影响代码逻辑,但在版本管理和发布审计中会造成混乱。建议修正为当前实际提交日期。

改进建议总结

  1. 修正日期:请将日期修正为实际提交日期,避免使用未来时间。
  2. 完善描述:建议将第一条变更描述补充得更具体,例如:
    * perf: remove wallpaper blur and effect features (replaced by ImageBlur1)
    * perf: migrate ImageBlur1 implementation to remote DBus service
    
  3. 标签优化:如果这次改动主要是架构调整,考虑将 perf 改为 refactor,除非有明确的性能测试数据支持性能提升的说法。
  4. 依赖声明:既然移除了内部功能并依赖 DBus 调用,请确保 debian/control 文件中的 DependsRecommends 字段已经更新,包含了提供 ImageBlur1 DBus 服务的包。

@deepin-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: mhduiy, wjyrich

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

@wjyrich wjyrich merged commit af3ef66 into linuxdeepin:master Apr 24, 2026
17 of 19 checks passed
@deepin-bot
Copy link
Copy Markdown
Contributor

deepin-bot Bot commented Apr 24, 2026

TAG Bot

Tag created successfully

📋 Tag Details
  • Tag Name: 6.1.87
  • Tag SHA: b7f6d4a2c031e8c3ed5ed7d803588d5eaf8962a2
  • Commit SHA: 582852b40c04099ec8bb05fe822108c303a80ffd
  • Tag Message:
    Release dde-daemon 6.1.87
    
    
  • Tagger:
    • Name: mhduiy
  • 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