主题正文#12 个月前
那次改动本身问题很多,我在评审里留了二十多条意见,还把其中一段截图发到小群里吐槽。后来代码改好了,写代码的人却有很长一段时间不再主动找我讨论。
我当时觉得自己是在守质量,现在看,意见大部分没错,处理方式很差。我把风格、风险和个人偏好混在了一起,也没先问他为什么这么写。更糟的是,小群截图让一次技术分歧变成了被围观。
后来我道了歉,也约定评审里把“必须改”和“建议”分开。关系有恢复,但不会当作什么都没发生。
想问做评审比较多的人:怎样既不放过问题,又不让评审变成展示资历?
那次改动本身问题很多,我在评审里留了二十多条意见,还把其中一段截图发到小群里吐槽。后来代码改好了,写代码的人却有很长一段时间不再主动找我讨论。
我当时觉得自己是在守质量,现在看,意见大部分没错,处理方式很差。我把风格、风险和个人偏好混在了一起,也没先问他为什么这么写。更糟的是,小群截图让一次技术分歧变成了被围观。
后来我道了歉,也约定评审里把“必须改”和“建议”分开。关系有恢复,但不会当作什么都没发生。
想问做评审比较多的人:怎样既不放过问题,又不让评审变成展示资历?
我会先判断能不能当面十分钟说清。评论超过十条时,继续在页面上追加通常只会让双方越来越防御。
“必须改”和“建议”很有用,但负责人也要克制建议数量。建议写了二十条,对新人来说还是二十条命令。