Skip to content

chore: Update version to 6.5.50#461

Merged
lzwind merged 1 commit into
linuxdeepin:masterfrom
add-uos:master
May 14, 2026
Merged

chore: Update version to 6.5.50#461
lzwind merged 1 commit into
linuxdeepin:masterfrom
add-uos:master

Conversation

@add-uos
Copy link
Copy Markdown
Contributor

@add-uos add-uos commented May 14, 2026

  • update version to 6.5.50

log: update version to 6.5.50

Summary by Sourcery

Bump the deepin-editor package version metadata to 6.5.50.1 across packaging files.

Build:

  • Update linglong packaging configuration to reference version 6.5.50.1.

Chores:

  • Align package versioning for the 6.5.50.1 release.

- update version to 6.5.50

log: update version to 6.5.50
@github-actions
Copy link
Copy Markdown

TAG Bot

TAG: 6.5.50
EXISTED: no
DISTRIBUTION: unstable

@sourcery-ai
Copy link
Copy Markdown

sourcery-ai Bot commented May 14, 2026

Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

Bumps the application package version from 6.5.49.1 to 6.5.50.1 in the linglong configuration, with a corresponding Debian changelog update stubbed in the diff.

File-Level Changes

Change Details Files
Update the application package version to 6.5.50.1.
  • Change the package version field from 6.5.49.1 to 6.5.50.1 in the linglong package manifest.
  • Ensure downstream packaging (Debian changelog) reflects the new version, as implied by the PR description.
linglong.yaml
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

你好!我是CodeGeeX。我已经仔细审查了你提供的 Git Diff 内容。

本次提交主要包含了版本号的升级(从 6.5.49 升至 6.5.50),以及相关的 Debian Changelog 和 Linglong 打包配置文件的更新。同时,Changelog 中提到了关于性能优化的两项改进:懒加载提升文件打开性能和懒加载标签页恢复加速启动。

以下是我对本次代码变更的详细审查意见:

1. 语法与逻辑

  • Debian Changelog 格式debian/changelog 的修改符合 Debian 规范,版本号、优先级、签名格式和时间戳均正确无误。
  • Linglong 配置格式linglong.yaml 中的版本号从 6.5.49.1 更新为 6.5.50.1,YAML 语法正确,缩进无误。
  • 版本号一致性debian/changelog 中的版本号为 6.5.50,而 linglong.yaml 中为 6.5.50.1。这在不同的打包系统中是常见的(Linglong 可能需要额外的构建版本号),逻辑上没有问题,但需确保这是项目预期的版本号管理策略。

2. 代码质量

  • Commit 规范与 Changelog 一致性:Changelog 中使用了类似 Conventional Commits 的前缀(chore:, fix(perf):, wip:),这有助于自动生成变更日志,质量很高。
  • ⚠️ WIP(Work In Progress)标记:Changelog 中出现了 wip: lazy tab restoring for faster startup将 WIP 状态的功能写入正式的版本发布 Changelog 中是不规范的。如果版本 6.5.50 是准备发布的正式版本,那么包含未完成的 WIP 功能可能会给用户带来不稳定的风险,也会让版本历史变得模糊。建议在发布前移除该条目,或者确保该功能已经开发完毕并将 wip 前缀移除。

3. 代码性能

  • 本次 Diff 未涉及核心代码:本次 Diff 仅为版本号和配置文件的变更,未包含实现“懒加载提升文件打开性能”和“懒加载标签页恢复”的实际 C++/QML 代码。
  • 针对 Changelog 描述的建议
    • 懒加载是提升编辑器启动性能的极佳手段。但在实际代码实现时,请务必注意内存生命周期管理异步加载时的线程安全
    • 在实现“懒加载标签页恢复”时,建议确保在标签页未完全加载前,用户界面的状态(如光标位置、滚动条位置、语法高亮)能够被正确缓存,并在加载完成后无缝恢复,避免界面闪烁或抖动(这与 6.5.49 修复的抖动问题相关联)。

4. 代码安全

  • 时间戳异常(未来时间):Changelog 中的时间戳为 Thu, 14 May 2026 16:26:09 +0800,这是一个未来时间。虽然这不会导致编译或打包失败,但在某些严格的 CI/CD 流水线或安全审计脚本中,未来时间戳可能会触发警告甚至构建中断。此外,这也会导致版本控制历史的时间线混乱。建议修正为当前真实的提交时间。
  • 打包版本防篡改:版本号提升是软件发布的关键节点。确保本次版本号更新是由授权人员(如 zhanghongyuan)在受保护的分支(如 main/release 分支)上进行的,避免未经授权的版本号覆盖或供应链混淆。

💡 改进建议总结

  1. 移除或更新 WIP 项:如果 6.5.50 是正式发布版本,请将 wip: lazy tab restoring... 更新为已完成的状态(如 feat(perf): implement lazy tab restoring...),或者从 Changelog 中删除该条目。
  2. 修正时间戳:将 Thu, 14 May 2026 修改为当前的真实日期,例如 Thu, 14 May 2024(假设为今年)。
  3. 补充核心代码审查:由于本次 Diff 仅涉及发版配置,建议将实现懒加载的 C++/QML 代码也提交审查,以便评估性能优化的实际效果和潜在的安全/稳定性风险。

修正后的 Changelog 示例:

+deepin-editor (6.5.50) unstable; urgency=medium
+
+  * chore: Update version to 6.5.50
+  * fix(perf): improve file opening performance with lazy loading
+  * feat(perf): implement lazy tab restoring for faster startup
+
+ -- zhanghongyuan <zhanghongyuan@uniontech.com>  Tue, 14 May 2024 16:26:09 +0800
+

希望这些审查意见对你有所帮助!如果需要针对具体的懒加载实现代码进行审查,请随时提供。

@deepin-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: add-uos, 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 595d340 into linuxdeepin:master May 14, 2026
21 of 23 checks passed
@deepin-bot
Copy link
Copy Markdown
Contributor

deepin-bot Bot commented May 14, 2026

TAG Bot

Tag created successfully

📋 Tag Details
  • Tag Name: 6.5.50
  • Tag SHA: a0684ce556288b9e6809908c07cc9191c0889d88
  • Commit SHA: 4de660c5b69d3c500b71e477bb36ccd7b3449d4d
  • Tag Message:
    Release deepin-editor 6.5.50
    
    
  • Tagger:
    • Name: add-uos
  • 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