代码审查
最后更新于
最后更新于
在 Bitwarden，我们鼓励每个人都参与代码审查。团队将主要关注他们自己的代码审查，但如果您看到一些有趣的内容，请随时加入并讨论。
要进行高效的代码审查，需要记住以下几点（来自 ）：
拉取请求应该是可管理的。如果 PR 太大（明显超过几百行），请询问贡献者是否可以在审查之前将其拆分为多个 PR。
这在我们的代码库中可能很棘手，因为许多事情都是紧密耦合的。
审查时请花些时间 - 预计每小时代码量少于 500 行。
休息一下——审查时间不要超过 60 分钟。
不要因为花时间进行代码审查而感到难过！它们通常花费的时间比您想象的要长，我们应该根据需要花费尽可能多的时间。
在开发周期早期发现的错误或缺陷，其修复成本要小得多。
如果您认为其他人对您正在审查的代码非常了解，请随时与他们联系或将他们添加为审查者。
请不要批准您不理解其含义的代码。随时欢迎提出意见和关注！例如，可以留下一些一般性评论或反馈，同时也表示您没有足够的知识来批准更改。作者可以要求其他人再次审查，并且 PR 上有两个审查者并没有什么问题。
请正确使用审查状态。
评论是讨论事物而无需明确批准或请求更改的好方法。
当您认为在合并 PR 之前需要更改某些内容时，应使用请求更改，因为这会阻止其他人在解决您的问题之前批准 PR。
我们应该毫不犹豫地使用此状态，但是我们应该就 PR 获得批准需要进行哪些更改提供明确的反馈。同样，PR 作者不应因更改请求而气馁，这只是表明应在合并 PR 之前进行更改。这很常见。
虽然可以放弃审查，但应谨慎使用。只要有可能，请先联系审查者，以确保他们的疑虑得到解决。以下是一些通常认为放弃审查的情况。
审查者离开办公室的时间较长，他们最初的反馈已得到解决。新审查者有责任确保原始反馈得到解决。
PR 是一个需要快速部署的修补程序，并且审查者处于不同的时区。如果反馈尚未得到解决，则可以在后续 PR 中解决。
批准 PR 意味着您对代码的工作原理以及它执行 PR 所声称的功能有信心。这可以基于测试更改或以前的领域知识。
PR 针对正确的分支。
您已验证链接的 Jira 议题描述是否与 PR 中所做的更改匹配。
您已阅读并理解 PR 建议的更改的全部影响。
您证明这些变化
解决了明显的问题，
以最好的方式解决了需求，
代码结构良好，
遵循了我们最新的、公认的模式，
并且没有意外的副作用。
如果您对上述任何内容不确定，请考虑使用不同的状态或先与作者联系以做讨论。另外，请毫不犹豫地请求其他人进行第二次审查。
如果 PR 影响多个团队，则需要所有受影响团队的批准。生成 PR 的团队（或者管理 PR 的团队（如果它源自社区））的审查者负责批准整个更改，而受影响的团队仅负责他们的代码库部分。
没有一种放之四海而皆准的代码审查技术。然而，有一些技术、工具和其他资源可以帮助您更有效地审查代码。
将代码审查分为多个重点领域会很有帮助。并一次专注于一个视角。
宏观视角——关注整个 PR。
问题解决了吗？
通过在适当的地方使用适当的抽象进行更改，是否可以有效地解决该问题？
PR 是否改变了您期望改变的领域？
有缺失的吗？
有没有意想不到的收获？
微观视角 - 专注于单个文件。
是否遵守代码样式？
代码可读吗？
是否遵循了以前的模式？
以前的模式仍然是正确的选择吗？
GitHub 界面有一些方便的工具可以帮助您审查代码。有关更多信息，请参阅下面的内容。
对多行进行评论
建议作者通过 GitHub 界面对代码更改立即接受并合并
小心这个！您没有享受到 IDE 的好处。这很容易破坏语法或格式。
隐藏空格（如果有大量缩进更改，则非常有用）
分别显示旧代码和新代码（这样您就可以在没有任何干扰的情况下阅读新代码） - 或将它们组合起来（这样您可以准确地看到发生了哪些更改）
许多更改可以在 GitHub 上在线查看。然而，有时在本地运行代码有助于提高您的理解 - 例如：
使用 IDE 功能（例如跳转到定义或查找引用）
重现您认为在代码中发现的错误
运行解决方案以了解它们如何组合在一起（宏观视角）。
要在本地运行代码，我们建议使用 GitHub CLI。这使您可以直接签出 PR，而无需管理远程分支 - 例如：
。如何评论 PR，包括：
。查看差异的不同方式，包括：