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

chore: remove unused comment #1024

Merged
merged 1 commit into from
Dec 20, 2024
Merged

Conversation

deepin-mozart
Copy link
Contributor

Log:
Change-Id: I64e12a5810ed98d6f8e1ddd1c86f2d71faa14986

Log:
Change-Id: I64e12a5810ed98d6f8e1ddd1c86f2d71faa14986
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: deepin-mozart

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

@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • 注释掉的代码可能包含敏感信息,应确保不会泄露到版本控制系统中
  • 如果注释掉的代码是用于初始化容器的,应考虑是否有必要保留这些初始化步骤

是否建议立即修改:

@deepin-mozart deepin-mozart merged commit 67fb2cf into linuxdeepin:qt Dec 20, 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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants