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

jsun2s 发表于 2023-02-22 21:41:30
comments可以分成blocking的和non-blocking的。non-blocking的可以找个google doc都记下来告诉他之后慢慢改。重点是一定找个地方记下来，省得日后说你不valu

Hmmm 非常不赞同这个想法 很多东西都是碰了 你新写的东西有style的问题那就该就是了
就算多做点refactor就行了 要是所有的小细节都要反问 能不能以后再改 ehh 我觉得非常的不make sense而且我会对这个人印象很不好

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

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

jsun2s 发表于 2023-2-22 21:41
comments可以分成blocking的和non-blocking的。non-blocking的可以找个google doc都记下来告诉他之后慢慢改 ...

这些回复作为reviewer看着挺不舒服的。
初来乍到，改个变量名这种花不了几分钟，没必要留下一个不合作的印象。
合理的要求你可以问能不能留个todo下一个PR改。当你说出is this critical的时候我相信大部分comment都不是那么critical，这相当于跟人说不critical的我不想改。那reviewer还能说啥，不愿意改，不合作呗。

主要还是要提高姿势水平，先建立trust，再逐步不卑不亢合理反驳。当reviewer认为你的姿势跟他在一个平面的时候，comment就成了交流意见，而不是指导。

offline sync一下

楼主也没说说具体都是啥commment。Coding style这种comment一般都是人家写一个就指望你把所有的类似问题都改了。楼主是不是没有看出来类似的hint？

你要反思一下，你老大已经对你的PR质量失去了信心！这是个危险的信号，一般都是会仔细看不太行的选手！
信任的人的pr就是快速一看！

jsun2s 发表于 2023-02-22 21:41:30
comments可以分成blocking的和non-blocking的。non-blocking的可以找个google doc都记下来告诉他之后慢慢改。重点是一定找个地方记下来，省得日后说你不valu

你这是要害死人家！
. 对着老板这么硬刚能有好结果！
不过生活中真的很多这种傻子……下场当然也是最好也就是不被pip

我一般前三个月会给人非常认真的看，因为前三个月的PR一般是以了解组内代码和建立组里的code style为主，速度慢也没事。这段时间如果代码的改动牵涉到其他地方我都会让他把那边也看了。可能会来来回回很多次，比如如果他没有看到我预估的程度，或者没有举一反三的把别的也覆盖到，

后面就随便看看了。能学会的都学会了，学不会的也就那样了。

你有两个选择：disagree and commit OR leave

匿名用户 发表于 2023-02-22 21:20:48
如果是一次性的comments的话，改的问题不会很大，但主要是pr总是被要求改了又改，不一次性留全。。要是之前的pr没merge的话，会影响之后的pr，这就导致进度被压下来了。。performance

对manager 而言你刚来肯定是规范更重要，不然你养成了这种习惯以后更麻烦