Warcraft III 拉取请求礼仪指南

Source: Dev.to

警告与错误

• “Can’t build there” – 当构建失败或代码放错位置时。
• “Work complete” – PR 已批准并合并。

混乱之中（开发者的生活）

• “What?” – 看见那段毫无意义的遗留代码。
• “Me busy!” / “Leave me alone!” – 当有人在我专注时请求审查。
• “No time for play!” – 正在生产环境修复 bug。
• “Me not that kind of Orc!” – 当有人建议使用技术变通方案时。

接受挑战

• “Ready to work!” – 打开一个新 PR。
• “Something need doing?” – 在队列中寻找待审查的 PR。
• “I can do that!” / “Be happy to!” – 实施审查者的建议。
• “Work, work!” – 编写下一个任务的代码。
• “Okie dokie!” – 在批准前的最后一条评论。