codecamp

MorJS 尊重大家的时间

一个好的代码审查过程从尊重时间开始。理想情况下,你希望在首次提交后两小时内开始审查代码。这主要是为了尊重提交 PR 的人的工作。

例如,你的一位同伴要求你审阅他们的工作,他们可能会等待你审阅几分钟。但如果审查没有迅速完成,他们将开始处理其他事情。

你的同伴将创建一个新的功能分支并开始为新任务编写代码。如果在四个小时后,你查看他们的第一个代码并发现它有错误,你的同伴将不得不暂停他们现在正在做的事情来进行更改。像这样的上下文切换可能非常耗时。由于不同人的工作方式各有差异,他们可能需要相当一段时间才能在回归原始任务时恢复注意力。

如果几天或几周过去了,而你依然不检查代码,情况会变得更糟。这个时候,你的同伴估计连代码是干什么的都忘记了。让他们回忆之前代码时会浪费大量的时间,而且你的同伴将更容易引入错误,因为他们已经有一段时间没有处理该功能了。

始终尊重他人的时间和工作,努力成为最及时的审阅者,并能够意识到你的同伴等待审阅的时间非常宝贵。


MorJS Code Review 概要
MorJS 始终提供建设性反馈
温馨提示
下载编程狮App,免费阅读超1000+编程语言教程
取消
确定
目录

MorJS 指南

MorJS 基础用法

MorJS 配置

MorJS 编译相关配置

MorJS 进阶用法

MorJS Web开发

MorJS Web开发介绍

MorJS Web开发快速上手

MorJS Tabbar IOS 小黑条适配开关

MorJS 社区

MorJS 规范

关闭

MIP.setData({ 'pageTheme' : getCookie('pageTheme') || {'day':true, 'night':false}, 'pageFontSize' : getCookie('pageFontSize') || 20 }); MIP.watch('pageTheme', function(newValue){ setCookie('pageTheme', JSON.stringify(newValue)) }); MIP.watch('pageFontSize', function(newValue){ setCookie('pageFontSize', newValue) }); function setCookie(name, value){ var days = 1; var exp = new Date(); exp.setTime(exp.getTime() + days*24*60*60*1000); document.cookie = name + '=' + value + ';expires=' + exp.toUTCString(); } function getCookie(name){ var reg = new RegExp('(^| )' + name + '=([^;]*)(;|$)'); return document.cookie.match(reg) ? JSON.parse(document.cookie.match(reg)[2]) : null; }