-
Notifications
You must be signed in to change notification settings - Fork 60
chore: bump version to 6.0.28 update changelog #394
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
Conversation
as title Log: as title
Reviewer's GuideThe package version was bumped to 6.0.28 and the changelog was updated accordingly by modifying the 'debian/changelog' file. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
deepin pr auto review这个提交的改动主要是修复了一个问题,即在收到kill信号后,dde-blackwidget会直接退出,从而解决了重新启动时卡logo的问题。从提交信息来看,改动是合理的,但还有一些细节可以进一步优化。 代码审查意见
改进后的提交信息示例这个改进后的提交信息提供了更多的上下文信息,并且提到了新增的单元测试,有助于其他开发者更好地理解这次提交的内容和目的。 |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: fly602, yixinshark The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
TAG Bot TAG: 6.0.28 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @fly602 - I've reviewed your changes - here's some feedback:
- The version bump to 6.0.28 appears to only modify
debian/changelog; please verify if other files also require this version update.
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
as title
Log: as title
Summary by Sourcery
Chores: