Reverted a teammate's agent PR that broke main and now I'm the asshole?
一位开发者回滚了同事一个包含9400行代码的PR,因为该PR导致了预发布环境的结账接口出现500错误,尽管CI显示通过。该同事的PR中有一个空路径问题曾被coderabbit/claude标记但被忽略。在周一的站会中,该同事指责回滚者是“控制狂”,认为其应该先联系自己而不是直接回滚。回滚者对此感到困惑,质疑为了修复主分支而回滚一个破坏性更改是否现在被认为是无礼的行为。
- 发布
- 2026年9月7日 11:46
- 来源类型
- 开发者社区
- 档位
- 社区
- 信源状态
- 正常
时间以 UTC 显示
更多信息
They merged a 9400 line PR out of Cursor on Friday afternoon. CI was green. Twenty minutes later staging was throwing 500s on the checkout endpoint, so I reverted it and left a comment saying what broke. Monday standup they go "you could have just pinged me to fix it" and call me a control freak in front of the whole team. The part that gets me is coderabbit / claude had actually flagged the null path in review and they resolved the thread without changing anything, so I don't know what is supposed to catch this if not me. Is revert first and ask after actually rude now? I just wanted main to work.