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

feat: update changelog and version #124

Merged

Conversation

pengfeixx
Copy link

Update changelog and version

update changelog

Log:update changelog
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. 版本号更新

    • 在多个文件中(arm64/linglong.yamldebian/changeloglinglong.yamlloong64/linglong.yaml)都更新了版本号,这是一个好的做法,确保了版本的一致性。
  2. changelog 文件格式

    • debian/changelog 文件中,新版本的提交信息应该更详细地描述变更内容,例如新增功能、修复的 bug 等。当前只有 "New version 6.5.8" 的描述,建议补充具体变更内容。
  3. 文件权限

    • loong64/linglong.yaml 文件的权限被修改为 100755,这通常表示该文件是一个可执行文件。如果这个文件不应该具有执行权限,建议将其权限修改为 100644
  4. 文件路径一致性

    • 确保所有相关文件(如 arm64/linglong.yamldebian/changeloglinglong.yamlloong64/linglong.yaml)的路径和名称保持一致,以避免混淆。
  5. 版本号格式

    • 版本号 6.5.8.1 中包含了一个点号,通常版本号格式为 主版本号.次版本号.修订号,例如 6.5.8。如果 1 表示的是修订号,建议保持版本号格式的一致性。
  6. 文件内容一致性

    • 检查所有文件中的 package 部分是否在其他文件中保持一致,包括 idnamekinddescription
  7. 注释和文档

    • 如果这些文件是配置文件或脚本的一部分,确保添加适当的注释和文档,以便其他开发者或维护者能够理解这些更改的目的和影响。

综上所述,代码审查主要关注版本号的一致性、文件权限的适当性、文件路径的一致性、版本号的格式规范以及提交信息的详细性。

Copy link

TAG Bot

TAG: 6.5.8
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: pengfeixx, wyu71

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

@pengfeixx pengfeixx merged commit 31cfc91 into linuxdeepin:develop/snipe Dec 20, 2024
15 checks passed
@pengfeixx pengfeixx deleted the update-changelog-20241220 branch December 20, 2024 06:17
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