如何处理审阅人的评论
当您发送了一个CL供评审时,评审人员很可能会对您的CL给出几个评论。下面是一些关于处理审阅者评论的有用信息。
不要认为这是针对个人的{#personal}
评审的目的是保持代码库和产品的质量。当一个评论家提供了代码的批判,认为它是试图帮助你,代码库,和谷歌,而不是人身攻击你或你的能力。
有时评论者感到沮丧,他们在评论中表达了这种沮丧。评论家这不是一个好的做法,但作为一名开发人员你应该做好准备。问问自己,“审稿人想要和我交流的有建设性的东西是什么?”
永远不要对代码评审评论做出愤怒的回应。这严重违反了职业礼仪,将永远存在于代码评审工具中。如果你太生气或生气而无法友好地回复,那么离开电脑一会儿,或者做点别的事情,直到你感到足够平静,可以礼貌地回复为止。
一般来说,如果一个评论者没有以建设性和礼貌的方式提供反馈,那就亲自向他们解释。如果你不能和他们面对面或视频通话,那就给他们发一封私人邮件。用一种友好的方式告诉他们你不喜欢什么,你希望他们做什么。如果他们也以非建设性的方式回应这个私人讨论,或者它没有预期的效果,那么根据情况升级到你的经理。
修复代码{# Code}
如果审阅人员说他们不理解代码中的某些内容,您的第一反应应该是澄清代码本身。如果无法澄清代码,则添加一个代码注释,解释为什么有代码。如果一条注释看起来毫无意义,