📣 Back to School开学季 - VIP通行证5折优惠!蓝莓、Offer多多同步优惠
12
返回列表 发新帖
楼主: 匿名
跳转到指定楼层
上一主题 下一主题
收起左侧

[同事协作] 这种情况该不该跟另一个senior正面刚

 
🔗
chersquare 2021-9-15 04:33:04 | 只看该作者
全局:
我这里说的你们可能已经想过了: 把他的CL提交了, 你的branch里先把那个commit revert然后rebase, 然后等你的做好了再提交; 这样你们短期内有一个不完整的补丁, 长期会有一个完整的解决方案, 你也不需要重写你的branch, 感觉不存在什么冲突啊?
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-O9HCD  | 添加认证 | 2021-9-15 04:53:15
没有人想自己辛苦些的代码要被放弃掉。你说的太直接,他肯定不高兴。如果是死脑筋白人,我见过很多跟你杠到底的。
公事公办就是你decline他的review,说不能根本解决问题,而且你已经在解决了。私下协商的好,就让他主动撤掉,这是聪明人都会选的,两边都不会太难看。
也要看你是否得罪得起这个人,不行先让他提交,你再想办法改回来。因为这里已经确定你得罪不起他,改的时候,也不要太直接了,用复杂一点的手段改,不要直接删了他的
回复

使用道具 举报

全局:
匿名用户 发表于 2021-09-13 19:04:27 ..
谢谢您的回复,很有帮助,我现在的困惑是,我对于这个fix能修多少,又会造成多少新的问题并不是很清楚,因为我看来这个fix已经造成了新的问题。但是本身figure out这件事我不知道时间上值不值得,因
他这个fix是基于一个existing bug,你作为refactor,在已经知道有这样的bug的情况下,你的版本必须要cover这个bug,不管是用他的fix还是你自己的fix solution。“不知道”以前还可以说不知道ignore掉,既然已经“知道”了,你anyway要了解这个bug以及fix
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-QKKYC  | 添加认证 | 2021-9-15 05:56:52 来自APP
建议lz避免和senior直接冲突,你们两个可以试着找一下有没有common ground
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 注册账号
职场达人
  • ↑ 本版用于讨论职场各种干货话题,闲聊请去🔗聊聊或者🔗匿名版
  • ❌ 本版严禁水贴,引战,发布广告,拉群,贴个人联系方式,扣分无警告
  • ☑ 求职、面经等去 🔗北美求职和 🔗回国求职大区,刷题和学习请去 🔗终身学习大区
  • ☑ 请去专版发布 🔗内推, 🔗招聘信息,和讨论 🔗创业内容
  • ☑ PIP / DevList/ Need Support 等话题也已开设 🔗专版

本版积分规则

>
快速回复 返回顶部 返回列表