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

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

屎山代码就是这么来的啊... 虽然代码和你无关, 但是你是这堆屎山的 owner, 还要天天在这堆 shit code 里工作, 这就和自己有关了啊
难道景区总有人随意大小便的解决方案不是 "不让人随意大小便 - 提供厕所等", 而应该是 "就让他们拉, 只要自己闻不到就行, 实在忍不了就去清理一下"???

我review代码都是和组员互相放水，然后过一段时间refactor一下codebase

我就喜欢别人留 comment，说明至少用心看了

感叹一下我们team的CR基本形同虚设。。。还挺想要有个senior 能够认真review的。。。

以前我们组的Senior维护代码质量的方式我觉得值得参考。

他在项目里引入JaCoco来保持一定的test coverage，这样每次测试不达标的代码就无法check in。如果是比较紧急的改动，可以接受下调coverage的比例。
在PR阶段，提出的comment基本修改一些比较明确会出错的点，如果是style、架构相关的话都有比较充分的理由。
愿意compromise，如果engineer指出相关改动的effort很大且投入产出比不高，就会主动放行。反馈及时，今日事今日毕，不会出现一条comment两天后回之类的情况。

这样的组代码质量不会差，开发进度也有保障，大家干得都很愉快

真的，你但凡做个人，别人至于绕开你吗？同样的hiring bar进来的，凭啥人家就能过，到你这里就是提出一堆问题？
根据过往的经验，所谓好的design，大多数时候只是凭空增加工作量。
我code review的原则，除非check in进去的代码会有bug，或者break lint，其他地方一律不block，最多给个nit，爱改不改。

我觉得选code reviewer肯定会有preference, 比如有人虽然严格，comments多，但是feedback快，这个我可以接受；最不爽的是，留了comments第二天就oof了

adddaa 发表于 2022-04-20 12:48:01
. 1point3acres真的，你但凡做个人，别人至于绕开你吗？同样的hiring bar进来的，凭啥人家就能过，到你这里就是提出一堆问题？
根据过往的经验，所谓好的design，大多数时候只是凭空增加工作量。
我code

不好意思，你这种啊，不敢苟同，什么叫做个人？人身攻击？你做个人也不会说出这种回复吧！你知道什么了说别人做个人了？我可以告诉你，team里面其他人都会让我review的，这个人新来的而已。

Ruby920421 发表于 2022-04-20 09:34:52
不会的！逐渐变成senior都要慢慢学会develop junior members。工作里沟通最重要。

好的，知道了，或者我至少可以提醒一下，involve service owner比较好吧。

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

和你说的这种情况一点关系都没有哈，我们review都非常及时的，几个小时之内肯定review完了。