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

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

想明哲保身只能做到又快又好

正常情况。你一个ng新人，manager亲自给你的PR很仔细地写comments，而且comments比较多，这说明了两件事。
第一，你的老板是个很负责的人。
第二，你还有很多需要学习和提高的地方。
好好珍惜吧，这对于新人来说，都是宝贵的财富。
另外，如果真的反复次数太多，严重影响了进度，你主动可以跟manager或tech lead约个会，把普遍的共性的问题一次性全解决掉。

还是那句话，对于新人来说，多学习，主动一些，准没错。

Love & Peace!

emmm zszszszs

说实话写code review是一件很累而且没什么credit的活。如果想找茬很容易在别的不容易发现而且又完全没办法的地方找。一般manager给你这么认真做code review是负责的表现。而且新来的人给你review也很重要，刚来的时候不好好养成习惯，找到正确的方法，日后问题只会有来越大。

新人很正常，一开始严格规范拉齐，之后轻松很多，很多是公司特定的规范，老员工都是知道的，所以一般不留comment。另一个则是信任问题，lz好好干，别一上来就发帖抱怨，赢得信任最重要，很正常。这算啥。

> 每次都留一堆comments，反而对内部同等级工作了一年的SWE不怎么留comments。

具体看看 the nature of comments. 是同事有类似的问题而不 comment 么? 还是 on the same criteria. 如果标准差不多的话, 多看看别人的 code review 尤其是有 meaty comments 的. 了解一定规律后 问问 teammate 能不能让你做 code review.

> 前几天的PR的改动其实都比较小，但老板总是会在变量名啊，数据结构啊，甚至和他相关的其他代码的结构加comments。

在 nits of variable names 上, 默认按 reviewer 的改. 自己刚写完, 思维一般在 optimized for code writer. 但 code 从 long-term maintainentance 上应该设计 optimized for code readers.

> 这就导致PR越写越长，好几天搞不定，还非常拖进度。因为之前听说有人因为交的PR太少而被pip的，所以也感觉这不是啥办法。

一个 PR/CL 代码的 scope 自己定好, 对于 out of scope request 明确 pushback 说明在 follow-up 解决.

>其实有一些comments我觉得能学到很多东西，包括可延续性，一致性和稳定性。但是有些给了反复好几天的comments无非是改变量名和记录每个attribute的详细信息

如果这些 design 概念对楼主 brand-new 的话, 建议去看完 comment 搜搜相关资料. 毕竟都是 hypothetical 的讨论. 真正理解, 注意下如果不这么做产生*不同*效果的例子.  注意, 没有 extensibility, consistency 这些的并不能说all bad. 取决于你写的东西的 lifecyle. 我曾经做过两了比较末端的项目, 完全是自主终端 isolated products, 从 launch 到 shutdown 都是我一个人搞定. 那这些 best pratices 很多就不受用, 更主要自己改起来舒服. 但大部分做 infra 的时候, 尽量 following the surrounding practices.

manager不会在针对你吧？

有的是故意找茬，但看下来 lz 这种应该90% 是 lz 自己的问题，coding style 不是随便说说或者什么表面文章，写code要想想 maintenance, best practice, 之后读你 code 的人会不会看不懂之类，觉得自己写的东西就是圣经、别人的code 都是屎山，真的不适合做这个工作

非常正常。 我和老员工一模一样的code。 我的那行被comment了需要改，老员工的code直接被approve了。就是一个earn trust的过程。
新进组的员工的code会被非常认真仔细得看，等你的code质量被认可了就不会那么多comment了。

> 楼主ng刚入职某公司不到一个月 这是我一个学习的机会要更注重代码质量吗？
这两个加起来几乎确定是你学习的机会，老板还挺有耐心。

* Don't take it personally. https://google.github.io/eng-pra ... dling-comments.html
* Always ask them their thoughts. https://hackernoon.com/the-art-o ... ts-prs-b520588eb345
* Review your own code first. https://leeorengel.com/review-comment-your-pull-requests/