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: bump version to 5.7.11 #642

Merged
merged 1 commit into from
Feb 27, 2025

Conversation

18202781743
Copy link
Contributor

update changelog to 5.7.11

update changelog to 5.7.11
@18202781743 18202781743 requested a review from mhduiy February 27, 2025 12:48
@deepin-ci-robot
Copy link
Contributor

deepin pr auto review

这个提交的changelog文件记录了几个修复,但需要进一步审查以确保其格式和内容正确。以下是具体的审查意见:

代码格式和风格

  1. 日期格式:提交日期应该使用标准的日期格式,例如YYYY-MM-DD。当前日期Thu, 27 Feb 2025 20:47:56 +0800不符合标准格式,建议改为2025-02-27
  2. 提交信息格式:提交信息应该简洁明了,并且遵循一定的格式。通常包括版本号、 urgency、更改内容以及作者信息。当前格式已经比较清晰,但建议检查是否有拼写错误或格式错误。

代码内容

  1. 修复描述:每个修复项的描述应该简短明了,并且能够清晰地说明修复了什么问题。例如,fix: window filickering caused by animation for DDrawer可以改为fix: 修复 DDrawer 动画引起的窗口闪烁问题,以更符合中文习惯。
  2. 版本号:确保版本号5.7.11是正确的,并且与实际修复的版本一致。

其他建议

  1. 一致性:确保整个changelog文件中的格式和风格保持一致,包括日期格式、提交信息格式等。
  2. 详细记录:如果可能,提供更详细的修复说明,包括修复的代码位置、修改的细节等,以便其他开发者更好地理解修复内容。

修改后的示例

dtkwidget (5.7.11) unstable; urgency=medium

  * 修复: DDrawer 动画引起的窗口闪烁问题
  * 修复: 在 Qt6 下 DBlurEffectWidget 应用崩溃的问题
  * 修复: 关闭 DAboutDialog 后标题栏无法拖动的问题
  * 修复: PrintPreview 无法切换页面方向的问题

 -- YeShanShan <yeshanshan@uniontech.com>  2025-02-27

通过以上修改,可以使changelog文件更加规范和易于理解。

Copy link

TAG Bot

TAG: 5.7.11
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, mhduiy

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

@18202781743 18202781743 merged commit 59839f9 into linuxdeepin:master Feb 27, 2025
19 of 21 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.

3 participants