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

fix: Remove deepin-service-plugin-network #232

Merged
merged 1 commit into from
Dec 27, 2024

Conversation

caixr23
Copy link
Contributor

@caixr23 caixr23 commented Dec 27, 2024

Remove deepin-service-plugin-network

pms: BUG-294423
pms: BUG-294395

Remove deepin-service-plugin-network

pms: BUG-294423
pms: BUG-294395
@deepin-ci-robot
Copy link

deepin pr auto review

这个提交移除了与deepin-service-plugin-network相关的所有代码和配置,包括DBus服务、安装文件、CMake配置、源代码文件等。这个改动可能会影响网络服务的功能,特别是与用户私有网络相关的功能。如果这个移除是故意的,那么需要确保相关的替代方案已经实现,并且所有依赖这个服务的组件都已经更新。如果这是误操作,那么需要撤销这个改动。

Copy link
Member

@BLumia BLumia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: BLumia, caixr23, fly602

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

@caixr23 caixr23 merged commit d118a25 into linuxdeepin:master Dec 27, 2024
9 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