Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update version 5.15.11 #555

Merged
merged 1 commit into from
Jan 10, 2025

Conversation

lzwind
Copy link
Contributor

@lzwind lzwind commented Jan 10, 2025

update version 5.15.11

Log: update version 5.15.11

update version 5.15.11

Log: update version 5.15.11
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lzwind

The full list of commands accepted by this bot can be found here.

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)都更新了版本号,这是一个好的做法,确保了版本的一致性。
  2. 版本号格式

    • 版本号从 5.15.10.1 更新到 5.15.11.1,这种格式通常用于软件版本管理,但需要确认是否符合项目的版本命名规范。如果规范是 主版本.次版本.修订号,那么这种格式是正确的。如果规范不同,需要调整版本号格式。
  3. changelog 更新

    • debian/changelog 文件中,新增了一行描述 qt6.8 version,这表明项目引入了 Qt 6.8 版本。需要确认这一变更是否与版本号更新相关,并且是否在代码中实现了相应的功能。
  4. 文件权限

    • loong64/linglong.yaml 文件的权限从 100644 更新到 100755,这通常意味着文件具有执行权限。需要确认这一变更是否必要,以及是否遵循了项目的文件权限管理规范。
  5. 文件一致性

    • 所有文件中的 version 字段都进行了更新,这确保了配置文件的一致性。但是,需要检查这些文件是否在同一个分支或仓库中,以避免版本不一致的问题。
  6. 注释和文档

    • debian/changelog 文件中,新增的变更描述 qt6.8 version 应该有更详细的注释,说明这一变更的具体内容、原因和影响。

总体来说,这些更改看起来是合理的,但需要确保它们符合项目的版本管理规范,并且所有相关的文档和配置文件都已更新。

Copy link

TAG Bot

TAG: 5.15.11
EXISTED: no
DISTRIBUTION: unstable

@lzwind lzwind merged commit b620e89 into linuxdeepin:develop/qt6.8 Jan 10, 2025
16 checks passed
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