-
Notifications
You must be signed in to change notification settings - Fork 24
审核君从入门到精通！
点击 https://github.com/dd-center/vdb/issues/new 向管理员介绍一下你自己
通过审核后你就拥有了编辑权限了=w=
上面有一行菜单栏，他们是干什么的呢？
[Code] 项目全部的代码
[Issue] 提问与回答专区
[Pull request] 代码提交与审核
[Action] 自动化检查与合并
后面没太大作用，略
点击[Pull request]，这里面是等待审核的用户
随便点开一个，往下翻，你将看到三个图标，每种图标有不同的状态
[人工复核] × Review required 需要人工复核 / ± Changes requested 代码没有过审 / √ Changes approved 代码通过审核
[代码检查] √ All checks have passed 自动检查无误（可能存在其他问题，见下文） / ◎ Some checks were not successful 代码有些问题
[冲突检查] × Merging is blocked 还不能合并代码 / √ This branch has no conflicts with the base branch 无冲突，可以合并 / △ This branch has conflicts that must be resolved 与其他代码有冲突，需更正
上述三项全部为√，代码才可合并
……等等，什么是“合并”？
【合并的简单理解】：用户提交的代码经过批准，被正式加入原程序
往上翻，找到[Files changed（有变化的文件）]选项卡，你可以在这里预览代码
【重要】你要审核以下内容：
1.用户是否达到收录标准，参考https://github.com/dd-center/vdb/issues/113
1.收录标准请参考https://github.com/dd-center/vdb/pull/11605/files
2.代码是否有误，尤其是"name"下的代码，这部分代码不能自动检查，请审核以下内容：
2.1.一对引号内只有一种语言/一个名字
2.2.同一行中，第一对引号内为语言，第二对为名字，不可空填！不能有空格！
2.3.语言大小写不论，均可，语言仅限英文字母
正确的示例：
"cn":"DD中心"
"en": "simon300000"
"JP": "DDs_moe"
错误的示例：
"cn": "" //空填
"simon300000":"EN" //倒位
"jp,kr":"DDs_moe" //多填
"中文"："DD_Center" //语言非纯英文
3.2.代码检查
这项检查将由系统自动完成，一般没有问题
出现问题可以参考日志解决，常见问题：1.社团不存在 2.文件名开头/结尾存在空格
对github不熟悉的新手可以不用理会此项报错，其他审核会协助解决
3.3.冲突检查
这项检查将由系统自动完成，一般没有问题
通常是两个人提交了同一个Vtb引起的
对github不熟悉的新手可以不用理会此项报错，其他审核会协助解决
一般情况下，代码已经打开[自动合并]，在同时满足以上三项时会自动合并。
前往[Files changed（有变化的文件）]，点击绿色按钮[Review changes（对变化进行评论）]，你会看到三个按钮
○ Comment 仅评论
○ Approve 批准代码并评论（可不填评论）
○ Request changes 驳回代码并评论
当你确认代码无误，用户有资格被收录后，请依次点击[Approve（批准）][Submit revew（提交评论）]来批准
请谨慎点击，一旦合并，此操作不可逆。下游服务器需要保证代码规范，如不确定代码是否有误，请请求其他审核协助。
经过此操作后，[人工复核]会变为通过状态。一般情况下，刷新网页后，请求会从 Open（开放）状态变为 Merged（已合并）状态
当你认为代码有误，但用户有资格被收录后，请点击[Review changes（要求修改）]，留下驳回理由，按下[Submit revew（提交评论）]来驳回
代码将等待修正之后才能合并
经过此操作后，[人工复核]会变为驳回状态。
当你确认用户不具备收录资格，请回到[Conversation（对话）]选项卡，点击下方[Close pull request（关闭请求）]按钮
经过此操作后，整个请求会被关闭。刷新网页后，请求会从 Open（开放）状态变为 Closed（已关闭）状态
有些时候，[人工复核]，[代码检查]，[冲突检查] 都通过，代码却没有自动合并，这是因为[自动合并]被手动关掉了。请回到[Conversation（对话）]选项卡，点击[Merge pull request （合并请求）]，[Confirm merge（确认合并）]来确认
请谨慎点击，一旦合并，此操作不可逆。下游服务器需要保证代码规范，如不确定代码是否有误，请请求其他审核协助。
至此，你已经是一名优秀的审核君了。