MorJS 尊重大家的时间
一个好的代码审查过程从尊重时间开始。理想情况下,你希望在首次提交后两小时内开始审查代码。这主要是为了尊重提交 PR 的人的工作。
例如,你的一位同伴要求你审阅他们的工作,他们可能会等待你审阅几分钟。但如果审查没有迅速完成,他们将开始处理其他事情。
你的同伴将创建一个新的功能分支并开始为新任务编写代码。如果在四个小时后,你查看他们的第一个代码并发现它有错误,你的同伴将不得不暂停他们现在正在做的事情来进行更改。像这样的上下文切换可能非常耗时。由于不同人的工作方式各有差异,他们可能需要相当一段时间才能在回归原始任务时恢复注意力。
如果几天或几周过去了,而你依然不检查代码,情况会变得更糟。这个时候,你的同伴估计连代码是干什么的都忘记了。让他们回忆之前代码时会浪费大量的时间,而且你的同伴将更容易引入错误,因为他们已经有一段时间没有处理该功能了。
始终尊重他人的时间和工作,努力成为最及时的审阅者,并能够意识到你的同伴等待审阅的时间非常宝贵。