Skip to content

chore: Bump version to 6.5.36#193

Merged
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
wangrong1069:pr0305
Mar 5, 2026
Merged

chore: Bump version to 6.5.36#193
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
wangrong1069:pr0305

Conversation

@wangrong1069

@wangrong1069 wangrong1069 commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

As title.

Log: Bump version to 6.5.36

Summary by Sourcery

Chores:

  • Update application version in linglong.yaml to 6.5.36.1.

As title.

Log: Bump version to 6.5.36
@github-actions

github-actions Bot commented Mar 5, 2026

Copy link
Copy Markdown

TAG Bot

TAG: 6.5.36
EXISTED: no
DISTRIBUTION: unstable

@sourcery-ai

sourcery-ai Bot commented Mar 5, 2026

Copy link
Copy Markdown
Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

This PR performs a routine version bump of the deepin-font-manager package to 6.5.36.1 in the Linglong packaging metadata, preparing for a new release.

File-Level Changes

Change Details Files
Update application/package version for new 6.5.36.1 release.
  • Change the package.version field to 6.5.36.1 in the Linglong manifest to reflect the new release version
  • Ensure packaging metadata aligns with the intended 6.5.36 tag in the release log
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

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

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

这段代码变更主要涉及 deepin-font-manager 项目的版本更新和构建配置文件的修改。以下是对该 diff 的审查意见,分为语法逻辑、代码质量、代码性能和代码安全四个方面:

1. 语法逻辑

  • 版本号一致性

    • debian/changelog 中将版本号更新为 6.5.36
    • linglong.yaml 中将版本号更新为 6.5.36.1
    • 问题:存在版本号不一致的情况。通常 linglong.yaml 作为容器构建(如 Linglong/玲珑)的配置文件,其版本号应与 debian/changelog 保持一致,或者遵循特定的映射规则。这里的 .1 后缀(构建号)如果是自动生成的,需要确认其逻辑是否正确;如果是手动维护的,需要确认是否与主版本号匹配。
    • 建议:请确认 linglong.yaml 中的 6.5.36.1 是否符合项目发布规范。如果 changelog6.5.36,通常 linglong.yaml 应为 6.5.36 或者 6.5.36.0(如果必须带构建号)。除非有特殊原因,否则主版本号应完全一致。
  • 日期异常

    • debian/changelog 中的日期为 Thu, 05 Mar 2026
    • 问题:年份为 2026 年,这显然是一个未来的日期,可能是系统时间错误或手动输入错误。
    • 建议:修正为当前正确的日期(例如 2024 年或 2025 年)。错误的日期可能会影响自动构建工具(如 dch)或发布流程的排序逻辑。

2. 代码质量

  • 提交信息规范

    • Commit Message 中使用了 chore: Unify linglong.yaml
    • 评价chore 通常用于构建过程或辅助工具的变动。这里的变动主要是版本号的更新,使用 chore 尚可,但 releasebump version 可能更准确地描述了这次变更的性质。
    • 建议:如果这次提交仅仅是为了发布新版本,建议使用 release: 6.5.36chore: bump version to 6.5.36,使变更意图更清晰。
  • 变更范围

    • Diff 显示修改了 changeloglinglong.yaml,旨在统一配置。
    • 评价:这符合版本发布的常规操作,确保了不同打包系统(Debian 包和 Linglong 包)的版本同步。
    • 建议:如果项目中有其他包含版本号的文件(如 CMakeLists.txt, meson.build, spec 文件等),建议一并检查并更新,以保持全项目版本号的统一。

3. 代码性能

  • 影响评估
    • 本次变更仅涉及元数据和版本号字符串的修改,不涉及任何源代码逻辑、算法或资源加载。
    • 结论:对程序的运行时性能(内存、CPU占用)和启动速度没有任何影响

4. 代码安全

  • 供应链安全
    • 版本号的正确更新对于软件供应链的追踪和漏洞管理至关重要。
    • 建议:确保 changelog 中记录了本次版本更新(6.5.35 -> 6.5.36)的具体内容。当前的 diff 只显示了一行 * chore: Unify linglong.yaml。如果这是一个正式发布版本,changelog 应该详细列出从上个版本修复的 Bug、新增的功能或安全补丁。仅仅写 "Unify linglong.yaml" 会掩盖版本发布的实际内容,不利于安全审计和用户了解更新内容。

总结与改进建议

  1. 修正日期:立即将 debian/changelog 中的日期从 2026 年修正为当前年份。
  2. 统一版本号:核实 linglong.yaml 中的 6.5.36.1 是否正确。如果只是为了跟随 changelog6.5.36,建议改为 6.5.36 以保持一致,除非 .1 代表特定的构建迭代且必须存在。
  3. 完善日志:在 debian/changelog 中补充 6.5.36 版本的实际更新内容(Bug 修复、新特性等),而不是仅保留构建配置的修改记录。
  4. 全面检查:检查项目中是否存在其他硬编码版本号的地方(如关于对话框、桌面文件等),确保它们都已同步更新。

@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

@wangrong1069

Copy link
Copy Markdown
Contributor Author

/forcemerge

@deepin-bot

deepin-bot Bot commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

This pr force merged! (status: unstable)

@deepin-bot deepin-bot Bot merged commit d75f427 into linuxdeepin:master Mar 5, 2026
18 of 20 checks passed
@wangrong1069 wangrong1069 deleted the pr0305 branch March 5, 2026 11:58
@deepin-bot

deepin-bot Bot commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

TAG Bot

Tag created successfully

📋 Tag Details
  • Tag Name: 6.5.36
  • Tag SHA: e4c6670eaf8c8b2fa927d1383dc8210867b82ba5
  • Commit SHA: 72aaa551f69030786f1d895c4a6298815547a37c
  • Tag Message:
    Release deepin-font-manager 6.5.36
    
    
  • Tagger:
    • Name: wangrong1069
  • 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