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: build faild #873

Merged
merged 1 commit into from
Aug 8, 2024
Merged

fix: build faild #873

merged 1 commit into from
Aug 8, 2024

Conversation

deepin-mozart
Copy link
Contributor

@deepin-mozart deepin-mozart commented Aug 8, 2024

remove unused header

Log:
Change-Id: Ie0a43f1b092bfeb150a6edf841903b7f7bb79a76

remove unused header

Log:
Bug:
Change-Id: Ie0a43f1b092bfeb150a6edf841903b7f7bb79a76
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • 移除了对QtXml库的引用,需要确认是否有其他代码依赖于此库,以防止潜在的编译错误。

是否建议立即修改:

  • 是,如果移除QtXml库会导致其他代码编译失败,应立即恢复该引用。如果确实不再需要,应确保相关代码已经更新以适应这一变化。

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deepin-mozart, Kakueeen, LiHua000

The full list of commands accepted by this bot can be found here.

The pull request process is described 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

1 similar comment
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deepin-mozart, Kakueeen, LiHua000

The full list of commands accepted by this bot can be found here.

The pull request process is described 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

@deepin-mozart deepin-mozart merged commit e7c10f1 into linuxdeepin:master Aug 8, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants