Skip to content

Conversation

@itsXuSt
Copy link
Contributor

@itsXuSt itsXuSt commented Jul 26, 2025

Updated .gitignore to exclude additional directories: .cursor, .cache, .claude, and build. Modified the formatDevice function in mainwindow.cpp to include a new option 'take-ownership' set to true in the QVariantMap.

Log: as above.

Bug: https://pms.uniontech.com/bug-view-325935.html

@github-actions
Copy link

TAG Bot

TAG: 0.0.1.18
EXISTED: no
DISTRIBUTION: unstable

Updated .gitignore to exclude additional directories: .cursor, .cache, .claude, and build. Modified the formatDevice function in mainwindow.cpp to include a new option 'take-ownership' set to true in the QVariantMap.

Log: as above.

Bug: https://pms.uniontech.com/bug-view-325935.html
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • .gitignore 文件中新增的忽略路径可能需要确认是否正确,避免忽略重要文件。
  • changelog 文件中的提交信息应该更加详细,描述具体修改的内容和原因。
  • mainwindow.cpp 文件中新增了 take-ownership 选项,需要确认这个选项的实现和安全性。

是否建议立即修改:

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: itsXuSt, max-lvs

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

@itsXuSt
Copy link
Contributor Author

itsXuSt commented Jul 26, 2025

/merge

@deepin-bot deepin-bot bot merged commit 88efff9 into linuxdeepin:develop/eagle Jul 26, 2025
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.

3 participants