Skip to content

chore: update version to 6.5.51#462

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

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

Conversation

@wyu71
Copy link
Copy Markdown
Contributor

@wyu71 wyu71 commented May 19, 2026

  • 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
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.

Sorry @wyu71, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@github-actions
Copy link
Copy Markdown

TAG Bot

TAG: 6.5.51
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot
Copy link
Copy Markdown

deepin pr auto review

你好!我是CodeGeeX。我已经仔细审查了你提供的Git Diff内容。本次提交主要涉及将 deepin-editor 的版本号从 6.5.50 升级到 6.5.51,并更新了相关的打包配置文件。

虽然这只是一个版本号更新的变更,但我在审查中发现了一个代码规范/格式问题和一个潜在的逻辑风险。以下是详细的审查意见:

1. 代码规范与格式

问题:debian/changelog 文件的末尾,删除了换行符。

- -- Ma Aiguo <imaiguo@gmail.com>  Wed, 02 Jun 2021 16:53:34 +0800
+ -- Ma Aiguo <imaiguo@gmail.com>  Wed, 02 Jun 2021 16:53:34 +0800
\ No newline at end of file

改进意见:
在Unix/Linux系统中,文本文件的标准规范是每一行都以换行符(\n)结尾。缺少末尾换行符不仅不符合POSIX标准,还可能导致某些命令行工具(如 catwcdiff 等)出现警告或异常行为,同时也可能影响某些打包工具的解析。
建议: 请在 Ma Aiguo 邮箱所在行的末尾恢复换行符,确保文件以空行结尾。

2. 逻辑一致性

问题: 版本号定义存在微小的逻辑不一致。

  • debian/changelog 中定义的版本是 6.5.51
  • linglong.yaml 中定义的版本是 6.5.51.1

改进意见:
虽然这可能是你们项目的既定规范(例如:主版本号为 6.5.51,而玲珑包的构建版本/修订号为 6.5.51.1),但需要确认这种版本号差异是有意为之还是疏忽遗漏

  • 如果是有意为之(例如最后的 .1 代表 linglong 包的构建号),则当前逻辑没有问题。
  • 如果两个文件应该保持绝对一致,则需要将 linglong.yaml 中的版本号修改为 6.5.51
    建议: 确认项目的版本号管理规范,确保各模块版本号语义的一致性。

3. 代码性能与安全

对于本次变更(仅修改版本号和更新日志),不涉及代码性能和代码安全方面的问题。


总结与修改建议

整体而言,本次变更非常简单明确,主要风险点在于文件末尾换行符的丢失。请在提交前补充末尾的换行符。

如果你使用的是 Linux 环境,可以通过以下命令快速修复换行符问题(在文件末尾添加空行):

echo "" >> debian/changelog

希望以上审查意见对你有所帮助!如果有其他代码需要审查,随时告诉我。

@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 e625d06 into linuxdeepin:master May 19, 2026
21 of 23 checks passed
@deepin-bot
Copy link
Copy Markdown
Contributor

deepin-bot Bot commented May 19, 2026

TAG Bot

Tag created successfully

📋 Tag Details
  • Tag Name: 6.5.51
  • Tag SHA: 150fea0338306a7794113db8361a971d56ef76dd
  • Commit SHA: 17cb8687c5ca8a2556a32b086e889396b9d93039
  • Tag Message:
    Release deepin-editor 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