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

[Deepin-Kernel-SIG] deepin: OWNERS: Add winnscode to reviewers #437

Merged

Conversation

Avenger-285714
Copy link
Collaborator

Bring in a real heavyweight expert to involve in our development.

Bring in a real heavyweight expert to involve in our development.

Signed-off-by: WangYuli <[email protected]>
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from avenger-285714. For more information see the Code Review Process.

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

关键摘要:

  • OWNERS文件中,Wenlpwinnscode的顺序被交换,需要确认这是否是预期的更改。
  • 检查是否有其他文件或配置也需要更新以反映这些更改。

是否建议立即修改:

  • 是,如果顺序的更改是无意的,那么需要立即修复以避免潜在的权限问题或混淆。
  • 否,如果这是有意的更改,并且已经更新了相关的配置文件和文档,那么可以不立即修改,但应该确保这些更改是充分的,并且已经得到了适当的审查。

@deepin-ci-robot
Copy link

The following users are mentioned in OWNERS file(s) but are untrusted for the following reasons. One way to make the user trusted is to add them as members of the deepin-community org. You can then trigger verification by writing /verify-owners in a comment.

  • winnscode
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.

@Avenger-285714 Avenger-285714 merged commit 2a48efc into deepin-community:linux-6.6.y Oct 7, 2024
2 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants