manager一直拖着pr的进度每次都给了很多comments怎么办

楼主ng刚入职某公司不到一个月，最近遇到老板反复push back PR，不一次性留全，每次都留一堆comments，反而对内部同等级工作了一年的SWE不怎么留comments。

前几天的PR的改动其实都比较小，但老板总是会在变量名啊，数据结构啊，甚至和他相关的其他代码的结构加comments。这就导致PR越写越长，好几天搞不定，还非常拖进度。因为之前听说有人因为交的PR太少而被pip的，所以也感觉这不是啥办法。

其实有一些comments我觉得能学到很多东西，包括可延续性，一致性和稳定性。但是有些给了反复好几天的comments无非是改变量名和记录每个attribute的详细信息，但这些在design doc里都有写，老板也没有提到。请问职场老手，这是我一个学习的机会要更注重代码质量吗？还是manager太注重细节，有点找茬？

我真的有一点分不太清，本身很简单的活现在变得要加班加点去改有点难受。

咱就是说。。。有没有一种可能。。。就是PR 里需要改的问题真的太多了。。

我也遇到过PR 留好多次comment 的情况。。不是不能一次留完comment, 是实在问题太多，实在看不下去了，想着先改干净一点再说。。。
或者提了几个问题，期待你能举一反三，把其他诸如格式结构的问题一起改了，然后发现你并没有一起改，三没有反出来。。只能继续留comment

听起来也没有很过分，只是lz不想去改。

匿名用户 发表于 2023-2-22 21:20
如果是一次性的comments的话，改的问题不会很大，但主要是pr总是被要求改了又改，不一次性留全。。要是之 ...

楼主你这个观点千万别在老板面前提出来。performance是不是会更注重项目的进度而可以牺牲一些质量？ 绝对会惹出大问题。

层主工作的组就是那种任何时候都不能忽视质量的。只要犯错，马上就是线上的客户问题，on-call的一大堆问题。很多人要跟着擦屁股。所以任何时候不能说牺牲质量。

只要在你老板手下做，就需要符合老板的做法。不然早晚跑路。所以尽量调整自己做事情风格去适应下

chen910 发表于 2023-02-22 21:15:09
听起来也没有很过分，只是lz不想去改。

如果是一次性的comments的话，改的问题不会很大，但主要是pr总是被要求改了又改，不一次性留全。。要是之前的pr没merge的话，会影响之后的pr，这就导致进度被压下来了。。performance是不是会更注重项目的进度而可以牺牲一些质量？楼主只想在裁员潮流下明哲保身，所以有点不太清楚该如何和manager就这件事相处。

约个1:1一起看PR，讨论一下。否则是非常浪费时间来来回回的。

正常 对新入职的都会注重coding style。

comments可以分成blocking的和non-blocking的。non-blocking的可以找个google doc都记下来告诉他之后慢慢改。重点是一定找个地方记下来，省得日后说你不value别人的feedback。

比如他说bad naming，但凡你遵守了code base的naming convention，就可以有底气的问他: “is this something critical that I have to address in this PR?”

再比如他所认为的better engineering问题，问他 “I don’t see any immediate need that we should do it now, am I missing anything?”

问一下组里有没有规范文档
二是看看这些comments make sense不 但是一个地方不能改两遍
三是找别人review 这种没啥意义

My two cents:
1. Don’t assume, ask. 既然你是新人。可以把你的疑问提出来 然后问。 恶意推定不利于建立trust.
2. Try you best to align your mindset with your manager’s priority At least in the next few months. You might see something different perspectives.
3. If you know your mgr is a jerk. Run ASAP.

楼主还年轻，其实正常的代码review就是这样子的。

要成长，很多人觉得事情一定要快，pr要快，“技术成长”要快，才能往上走。其实很多时候快不快根本不是你说了算的。职场老手相对新手一个能力在于能够比较准确的判断一个工作需要多少时间，以及能力和资源去保证这个工作在预估的时间内完成。简单点说，manager不怕你慢，怕你天天给人surprise, 说了三天干完拖了十天，而且反复如此。你的manager审阅代码风格如此，组里的风气不可能是求快的。