-
Notifications
You must be signed in to change notification settings - Fork 23
审核君从入门到精通！
本文将教你如何在vdb审核代码！
右上角，[sign in]登陆，[sign up]注册
点击 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
2.代码是否有误，尤其是"name"下的代码，这部分代码不能自动检查，请审核以下内容：
1.一对引号内只有一种语言/一个名字
2.同一行中，第一对引号内为语言，第二对为名字，不可空填！不能有空格！
3.语言大小写不论，均可
正确的示例：
"cn": "simon300000"
"JP": "DDs_moe"
错误的示例：
"cn": "" //空填
"simon300000":"EN" //倒位
"jp,kr":"DDs_moe" //多填
3.2.代码检查
这项检查将由系统自动完成，一般没有问题
出现问题可以参考日志解决，常见问题：1.社团不存在 2.文件名开头/结尾存在空格
对github不熟悉的新手可以不用理会此项报错，其他审核会协助解决
3.3.冲突检查
这项检查将由系统自动完成，一般没有问题
通常是待审核的代码与原有代码不符引起的
举个例子：
原代码："cn": "DDs_moe"
新代码："JP": "DDs_moe"
对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]来确认合并
请谨慎点击，一旦合并，此操作不可逆。下游服务器需要保证代码规范，如不确定代码是否有误，请请求其他审核协助。
至此，你已经是一名优秀的审核君了。