Skip to content

Conversation

@yixinshark
Copy link
Contributor

as title

Log: as title

deepin-ci-robot added a commit to linuxdeepin/dde-session-shell-snipe that referenced this pull request Jul 8, 2025
Synchronize source files from linuxdeepin/dde-session-shell.

Source-pull-request: linuxdeepin/dde-session-shell#20
deepin-ci-robot added a commit to linuxdeepin/dde-session-shell-snipe that referenced this pull request Jul 8, 2025
Synchronize source files from linuxdeepin/dde-session-shell.

Source-pull-request: linuxdeepin/dde-session-shell#20
deepin-ci-robot added a commit to linuxdeepin/dde-session-shell-snipe that referenced this pull request Jul 8, 2025
Synchronize source files from linuxdeepin/dde-session-shell.

Source-pull-request: linuxdeepin/dde-session-shell#20
as title

Log: as title
deepin-ci-robot added a commit to linuxdeepin/dde-session-shell-snipe that referenced this pull request Jul 8, 2025
Synchronize source files from linuxdeepin/dde-session-shell.

Source-pull-request: linuxdeepin/dde-session-shell#20
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. .reuse/dep5文件中,新增的# cmake部分没有指定版本号,建议添加版本号以便于依赖管理。
  2. LICENSES/GPL-3.0-or-later.txt文件中,链接地址从http更改为https,这是一个好的实践,因为它增加了安全性。
  3. plugins/examples/login-plugins/login-udcp/qss/udcp_mfa_login_widget.qss文件中,版权声明和许可证标识已经正确添加,但建议检查是否有必要更新版权年份。
  4. src/pam-inhibit-autologin/pam_inhibit_autologin.c文件中,版权年份从2011 - 2025更改为2021 - 2022,确保这是最新的年份。
  5. src/session-widgets/fullmanagedauthwidget.hsrc/widgets/abstractfullbackgroundinterface.hsrc/widgets/popupwindow.cppsrc/widgets/popupwindow.h文件中,版权年份从2011 - 2022更改为2021 - 2022,确保这是最新的年份。
  6. src/widgets/popupwindow.cppsrc/widgets/popupwindow.h文件中,许可证标识从LGPL-3.0-or-later更改为GPL-3.0-or-later,需要确认这一更改是否符合项目的许可证策略。
  7. 所有文件的版权声明和许可证标识都已经正确添加,并且格式统一,这是一个好的实践。

总体来说,这些更改都是合理的,但需要确保它们符合项目的许可证策略和版权要求。

@deepin-ci-robot
Copy link

[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.

Details 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

@yixinshark yixinshark merged commit 6817b6e into linuxdeepin:master Jul 8, 2025
15 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