-
Notifications
You must be signed in to change notification settings - Fork 31
feat: 移除dde-login-reminder, 迁移到dde-session-ui中 #178
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
Reviewer's guide (collapsed on small PRs)Reviewer's GuideUpdates Debian packaging metadata to add qt6-tools-dev as a build dependency so the package has Qt6 development tools available during build. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
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.
移除dde-login-reminder, 迁移到dde-session-ui中 Log: 移除dde-login-reminder, 迁移到dde-session-ui中
deepin pr auto review这个diff显示了一个重大的代码变更:移除了整个dde-login-reminder模块。让我来分析这个变更:
建议改进:
这个变更看起来是一个重大的功能移除,可能会影响系统的安全性。建议仔细评估这个变更的影响,并确保有适当的替代方案来维护系统的安全功能。 |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: fly602, robertkill 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 |
除dde-login-reminder, 迁移到dde-session-ui中
Log: 移除dde-login-reminder, 迁移到dde-session-ui中