Code review故意找一个容易过的人看？

我很appreciate那种insightful comments
因为review很花时间 有那功夫自己写diff不好吗

可现实是，很多人留两个不痛不痒comment 加了blocking review
你回头找人两天不理你
好不容易回复了 又加两个comment
来来回回经常一周就没了

我耗不起这时间下次直接绕过这种sb 你爱跟谁玩谁玩
作为reviewer一点职业素养没有

想要effective 就要自己maintain一个list里面是跟你打交道的靠谱service owner/reviewer 否则你的project就等着被block吧 这也是重要技能之一

这些所谓的"严格"，经常是下意识地装逼

我就是review代码很严格，会提很多结构上的问题，比如把改动一小部分的class整个重构，也不是一定要求，就是提出建议

现在很少有人找我review代码。。。

匿名者 发表于 2022-4-20 06:56
只要能work就行，实在不爽就自己再改，在别人的change里面留一堆comments，别人也可以不爽
另外提交的code ...

还有这种工程师？兄弟麻烦说下你是哪个公司的？

匿名用户 发表于 2022-04-20 10:00:30
我很appreciate那种insightful comments
因为review很花时间 有那功夫自己写diff不好吗

对 block之后改掉很快approve的可以忍 有的人提了一堆有的没的然后就在那block 人就找不着了 特别烦

有人告诉我怎么改这不是挺好的吗。。
顺势问下为什么就能避免未来可能出现的差错了。。

1. 从 process 上, 可以看你们的 code review 系统能不能设置 team 为 reviewer, 至少自动 CC, 这样发出去的代码所有人 (至少你) 可以都在 review 阶段就看到. 如果真的必要, 直接设置必须再额外有 owner 的 approval.
2. 从文化上, 在 team 内推进 code style 等重视程度, 以及 design review meetings
3. 从 coding 上, 看能不能设置更好的 linter, presubmit check 等等. 机器给出的 feedback 会比 code review comment 更容易接受.
4. 从个人, 直接去 1:1 了解具体情况, 沟通有没有什么更好的办法. 你能不能给出更 general 的 coding 建议, 学习建议, 来避免总是出问题.
5. 从自己, 看看有没有一些可以放过的问题. 或者不是非常重要的问题就要求开个 bug, 然后 code 写个 todo.

团队合作就是这样子。根本问题出在你跟你现在公司的环境不匹配。你对service对code有高标准，但是其他人达不到，一旦你休假或者因为什么原因放手，代码质量就你的标准而言会直线下降。根据你的描述，虽然你是名义上的owner，但是其他人可以不需要你的review commit code，我觉得大概有下面三个办法，难度依次增加。
1. 你换去一个周围的人都跟你标准一致的地方，大家都很自觉甚至公司有强制的guidance维护质量。
2. 你降低标准，跟你周围的人一致。
3. 你制定标准，强制要求周围的人按你定的标准来。比如不经过你或者不经过某些自动系统就无法commit（也不一定就是完全按照你的，细节可以沟通但是大致差不太多）