Skip to content

Conversation

@tsic404
Copy link
Contributor

@tsic404 tsic404 commented Dec 13, 2024

No description provided.

make desktop in a separate process

log: as title
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. Changelog 文件:

    • changelog 文件中,新增的功能描述应该更加详细,例如 "feat: set a fixed cursor size on Wayland" 可以补充为 "feat: 在 Wayland 下设置固定的光标大小,以改善用户体验"。
  2. CMakeLists.txt 文件:

    • CMakeLists.txt 文件中新增的 install_symlink 调用应该有相应的注释说明其用途,例如 "安装 dde-shell-plugin 服务到 dde-session-initialized.target.wants"。
  3. dde-shell-plugin@org.deepin.ds.desktop.service 文件:

    • [Unit] 部分中的 Description 应该更加详细,例如 "Description=dde-shell 插件用于桌面环境,提供额外的桌面功能"。
    • ExecStart 行中,-p %I 参数可能需要解释其用途,例如 "ExecStart=/usr/bin/dde-shell -p %I - 插件模式启动 dde-shell"。
  4. dde-shell@DDE.service 文件:

    • ExecStart 行中的 -d org.deepin.ds.desktop 参数应该有相应的注释说明其用途,例如 "ExecStart=/usr/bin/dde-shell -C %I --serviceName=org.deepin.dde.shell -d org.deepin.ds.desktop - 指定桌面服务名称"。
  5. 代码风格:

    • 所有新增的 .service 文件中,[Unit][Service] 部分的键值对应该遵循统一的缩进和格式,以提高代码的可读性。
  6. 安全性:

    • 确保所有新增的 .service 文件中的 ExecStart 命令没有引入任何潜在的安全风险,例如未经验证的用户输入。
  7. 性能:

    • 检查新增的 dde-shell-plugin@org.deepin.ds.desktop.service 服务是否有必要在 dde-session-initialized.target.wants 中启动,以避免不必要的资源占用。
  8. 错误处理:

    • dde-shell-plugin@org.deepin.ds.desktop.servicedde-shell@DDE.service 文件中,应该添加错误处理逻辑,例如在 ExecStart 命令执行失败时,记录错误日志并采取相应的恢复措施。
  9. 依赖管理:

    • 确保所有新增的 .service 文件中列出的依赖项(如 dbus.socket)都已正确安装,并且版本兼容。
  10. 文档和注释:

    • 为新增的 .service 文件添加详细的文档和注释,以便其他开发者理解其用途和工作原理。

@github-actions
Copy link

github-actions bot commented Dec 13, 2024

TAG Bot

TAG: 1.99.5
EXISTED: yes
DISTRIBUTION: unstable

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: BLumia, tsic404

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

@deepin-bot
Copy link

deepin-bot bot commented Dec 14, 2024

TAG Bot

New tag: 1.99.5
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #108

@yixinshark yixinshark merged commit 64a0d7e into linuxdeepin:master Dec 14, 2024
17 of 18 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.

4 participants