同事在开发某模块时，因为我之前做过类似的功能，改写了我的某个方法（我当时建议他新建个 service ，不要直接动我的），然后发版之后导致我这边异常了（我们没有 code review ），然后用户找上来了，我问 cursor 我怎么回复，cursor 建议如下: "这次问题的 technical root cause 是代码变更没有经过 review ，我作为模块 owner 有监管责任。我和[同事]已经深入讨论过，制定了完善的预防措施，确保类似问题不会再发生"，这措辞让我感觉自己格局低了，真有人能这么大度吗？大家怎么看？没有啥经济损失，只是补数据要补蛮久，我准备到数据导给同事让他补，领导要问我就说他干的。
11 条回复 • 2025-07-29 09:51:26 +08:00
 |
1
czzt1 3 天前
没有 review 那就谁改的谁背锅
|
 |
2
yuanmomo 3 天前 via iPhone
不对啊，改了也没测试么）
|
 |
4
fenddddddda 3 天前  1
最终问题出在你现在的模块，你一定是有责任的。 最次也是个没有 review 的责任。
你直接说是因为其他人改出的问题，你领导一听 一定认为你是在甩锅。。。 你换个思路，你可以说 是对方没有按照你的指导，引发的问题。 突出在没有听你的方案上。。。。 |
 |
5
woodfizky 3 天前 1
没有 code review&没有完整测试用例做回归测试，个人理解主要问题在改你代码的人，次要责任在整个团队缺少 code review 环节&完善的回归测试，技术头头/组长没有制定相关制度&没有有效监督的问题。
个人感觉你别等领导问你了，你主动汇报一下情况，然后给点建议说抽时间做完善回归测试和 code review ，领导不采纳那是领导/时间预算问题了。 |
 |
6
erwin985211 3 天前 2
肯定是谁改谁背锅呀。要不然 bug 能可以归到构建项目的那个人身上了。
|
 |
7
coderluan 3 天前 1
你们连 review 都没有，那么领导不喜欢谁就是谁的锅。
|
 |
8
wesleywaters 3 天前
@coderluan 你说的才是真相
|
 |
9
wjx0912 3 天前
没有 review, branch 也没有 lock ，应该有自己的问题。另外谁编译的？谁发布的？
|
 |
11
yjxjn 2 天前
@erwin985211 可不，正常是开发完事，测试出测试 report 说 OK ，就可以走之后流程，我感觉这是测试貌似都没有？？
这不测试的问题吗？ 如果没测试的情况下，我们一般这么划分责任： 他没动你的代码前，你这代码有问题不？ ->有的话，你和他都有责任。 ->没有的话，他的责任。 |
Select Language