Skip to content

chore: Bump version to 6.5.17#333

Closed
rb-union wants to merge 1 commit intolinuxdeepin:masterfrom
rb-union:bump_version
Closed

chore: Bump version to 6.5.17#333
rb-union wants to merge 1 commit intolinuxdeepin:masterfrom
rb-union:bump_version

Conversation

@rb-union
Copy link
Contributor

@rb-union rb-union commented Mar 6, 2025

Bump version to 6.5.17

Log: Bump version to 6.5.17

Bump version to 6.5.17

Log: Bump version to 6.5.17
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: rb-union

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

代码审查意见:

  1. 版本号更新

    • 在多个文件中(arm64/linglong.yamldebian/changeloglinglong.yamlloong64/linglong.yaml)都更新了版本号从 6.5.16.16.5.17.1。这是一个合理的操作,但需要确保所有相关的文档和配置文件都已更新,以保持版本一致性。
  2. changelog 更新

    • debian/changelog 文件中,新增了一个条目来描述版本更新。这个条目包含了修复的 bug 和提交者信息。这是一个好的做法,有助于维护者跟踪版本变更和问题修复。
  3. 文件格式和一致性

    • 所有文件中的 linglong.yaml 文件格式和内容保持一致,这是一个好的实践,有助于维护代码的整洁和一致性。
  4. 注释和文档

    • debian/changelog 文件中,新增的条目没有包含具体的修复细节。建议在 changelog 中添加更详细的描述,以便其他开发者或维护者能够快速理解修复的内容。
  5. 文件权限

    • loong64/linglong.yaml 文件的权限从 100644 更新为 100755,这通常意味着文件具有执行权限。需要确认这是否是预期的更改,以及是否有必要为这个文件设置执行权限。

总体来说,这次提交的改动是合理的,但建议在 changelog 中添加更详细的修复描述,并确认文件权限的更改是否符合预期。

@github-actions
Copy link

github-actions bot commented Mar 6, 2025

TAG Bot

TAG: 6.5.17
EXISTED: yes
DISTRIBUTION: unstable

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Mar 12, 2025

TAG Bot

New tag: 6.5.17
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #334

@rb-union rb-union closed this Mar 17, 2025
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