-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
ci: appoint PR reviewers via action bot reviews #9913
base: master
Are you sure you want to change the base?
Conversation
3e75f0a
to
8d799f2
Compare
你这个要加RTTHREAD_GITHUB_TOKEN的吧 |
目前这个仓库里面的RTTHREAD_GITHUB_TOKEN 加了吗?还没有加吧? |
没有的,那个是我加的。token你找熊大加一下。 |
噢噢好的,那我另设一个 |
8d799f2
to
93f979c
Compare
不是我的意思是你还是用RTTHREAD_GITHUB_TOKEN 这个就可以了。这个settings里面的值没有赋值。 |
93f979c
to
6a39a62
Compare
@BernardXiong 熊大,麻烦帮忙添加个名为 GITHUB_TOKEN_COMMENT 的token可以嘛,只需要拥有对仓库的评论权限就可以了 |
拉取/合并请求描述:(PR description)
[
为什么提交这份PR (why to submit this PR)
Ps:由于使用CI机器人自动任命审核者需要被任命的审核者拥有仓库的一些读写权限,所以换一种方式来通知PR审核,通过CI自动识别PR中的label来任命审核人。
你的解决方案是什么 (what is your solution)
使用方式:维护者可以通过修改仓库目录下的 .github\label-reviewers.json 文件,通过修改对应的键值,可实现对审核者的通知:
例如:
CI脚本特性:
请提供验证的bsp和config (provide the config and bsp)
]
当前拉取/合并请求的状态 Intent for your PR
必须选择一项 Choose one (Mandatory):
代码质量 Code Quality:
我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:
#if 0
代码,不包含已经被注释了的代码 All redundant code is removed and cleaned up