Skip to content

Conversation

@BLumia
Copy link
Member

@BLumia BLumia commented Aug 19, 2024

由于 TreeLand 计划更新协议,暂时移除相关调用。

Log:

由于 TreeLand 计划更新协议,暂时移除相关调用。

Log:
@BLumia BLumia requested a review from justforlxz August 19, 2024 07:36
@deepin-ci-robot
Copy link

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: BLumia

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

@BLumia BLumia marked this pull request as ready for review August 19, 2024 07:36
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • 代码注释应该明确说明为什么暂时返回true,而不仅仅是“TreeLand plans to update the related protocol”。
  • 如果该函数的目的是为了防止在某些情况下调用父类的personalizeWindow方法,那么应该在代码中添加相应的逻辑来处理这种情况。

是否建议立即修改:

@deepin-bot
Copy link

deepin-bot bot commented Aug 22, 2024

TAG Bot

New tag: 0.8.6
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #406

@BLumia BLumia merged commit cc83136 into linuxdeepin:master Aug 23, 2024
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