新人求指点迷津

本人new grad第一份工作，工作了半年，组里只有我一个junior SDE。最近组里在写一个新的service，大家都有contribute。我前期负责research data怎么拿，后面负责servide里的一小块，每天辛辛苦苦忙碌，终于把code写出来了。后面有个事情我比较想不通，想请各位大佬指点。我的PR merge了以后，组里其他的同事就会raise另外一个PR，来改我的code，基本就是把我的code删掉，然后copy到另外一个地方然后加一些，改一些。Code review的时候我会质疑为什么要这样做，但是他说这个是要加这个那个的feature进去。Tech leader感觉也是无所谓，我也不好再刚。我明白大家应该协作，但是这样在code base看起来，没有几行是我写的了，很郁闷！我会经常纠结ownership的问题，大家有遇到这样的情况吗？如果这是什么套路，请大家指点。如果是我太玻璃心了，也请大家骂醒我！

可能是他觉得位置不对？不过位置不对的话review的时候可以提出来啊

shaw007 发表于 2023-04-24 14:16:01
可能是他觉得位置不对？不过位置不对的话review的时候可以提出来啊

他review的时候告诉我 他后续有一个PR 要integrate所有的function什么的。其实我觉得如果他让我自己改我这部分会更好。

待过fg，仅以个人经验认为，一般codebase没有owner，大家都可以改，但你作为reviewer可以不approve他的code change，如果你有合适的能说服他的理由。一般一个scope会有一个或多个owner，别人做code change一般要找owner review或者讨论，主要是因为owner对scope里的细节有最深的了解和规划，与owner沟通是确保code change以后不会出问题。从楼主描述来看因为比较junior，别人可能并不觉得你是scope的owner，或者对这个改动比较自信，或者这个改动本身比较trivial。总而言之建议楼主作为junior多从解决实际问题出发与别人讨论，想想别人这么改在这个service里是不是合适，如果不合适就勇敢提出你的想法，用逻辑和数据支撑你的想法，如果合适那就接受别人的改动。如果你觉得他的改动可有可无，也可以直接问他为什么，说不定能学到东西。你的贡献是code change和code review，不是最终repo里你的行数，不要过于担心。在mindset上多倾向于解决实际问题，少一些defensive，这是大部分人都会喜欢合作的类型。

匿名用户 发表于 2023-04-24 22:42:49
待过fg，仅以个人经验认为，一般codebase没有owner，大家都可以改，但你作为reviewer可以不approve他的code change，如果你有合适的能说服他的理由。一般一个scope会

谢谢大佬回复！非常有帮助，工作以来一直在经历mindset的改变。