此 PR 所做的更改是否必要？如果您对某个特定主题领域有深入的了解，您可能能够指出哪些更改对于实现既定目标而言是不必要的。所有代码都必须维护，这可能代价高昂。有时，代码审查可以实现的最有效的事情就是防止编写不必要的代码。
此 PR 是否重复了现有功能？如果您发现有人试图重复现有功能，请向他们指出现有解决方案。如果该解决方案不能完全满足他们的需要，请查看是否有办法为现有解决方案做出贡献，而不是自己开发。
还有其他人应该知道这个 PR 吗？由于代码审查往往 厄瓜多尔 whatsapp 电话号码列表 团队环境中进行，因此对其他团队来说很重要的 PR 被忽视的情况并不少见。花点时间标记您的联系人，让他们知道可能需要了解这个 PR。
进行更详细的审查
并非所有这些都适用于所有情况。请选择适用的。
可读性。可读性测试的目的是确保六个月后阅读代码的人能够快速建立代码的连贯心理模型。
对于那些对代码库几乎没有或根本没有经验的人来说，这种变化是否能够合理地理解？假设您了解该语言，但不了解代码库。您是否能够轻松理解所有内容？如果不是，为什么？
是否使用了任何深奥的语言功能？深奥的语言功能虽然偶尔有用，但往往会损害可读性，即使是语言专家也是如此。如果您看到深奥的语言功能被使用，请询问是否可以使用更简单的构造。如果不行，请确保对该功能进行注释或以其他方式记录以减少认知开销。
生产准备。生产准备通过后，我们可以确信代码是安全的、有文档记录的、可靠的，可以放心投入生产。
我们如何知道这段代码何时中断？通常，这个问题的答案是“如果出现未处理的异常，将触发包含堆栈跟踪的警报”。这是一个非常合理的答案。无论如何回答，您都需要一种方法来知道代码何时中断。如果您不需要知道它何时中断，您可以删除该代码。
此更改是否需要新的文档？文档通常会变得陈旧，因为它不会随其记录的代码一起发展。花点时间问问自己是否需要创建新的文档或是否需要更新现有文档。
是否有测试可以防止回归？大多数变更都应附带自动验证变更的方法。如果没有测试，则应解释原因。
这种更改是否安全？安全很难，并不是每个人都能成为安全专家。但是，大多数经验丰富的程序员应该至少知道一些使代码不太安全的应用程序编程模式。如果您编写数据库代码，您应该知道 SQL 注入漏洞是什么样子的。如果您编写前端代码，您应该知道 XSS 漏洞是什么样子的。如果您不确定，或者您正在进行潜在的风险更改，请在审核中标记您公司安全团队的某个人。
