新人PR一次1200+， 400-这个算多吗

RT，最近做feature的时候，涉及到改动很多前人的代码，这次PR我发现有1200+，400-


初入职场，请问这个是正常的量，还是太多了。


担心会让reviewer review起来比较难。而且之前也听前辈说，量大的话可以分多次PR，这样别人也好理解，出错了也容易rollback。但是不知道什么是适合的量？


真心感谢

直接pip了

150-150 rule: 指+/-diff都不超过150行。个人认为没必要非要按照150-150来发，但是它大体代表了一个对publisher和reviewer都比较友好的cr体量。像1200这种已经超过一个数量级，有点太大了

https://www.linkedin.com/pulse/n...ramanan-subramanian#:~:text=Code%20reviews%20that%20are%20large,harder%20to%20get%20quality%20feedback.&text=A%20good%20practise%20is%20to,review%20cycle%20in%20a%20day.

看到500+行的PR我直接血压升高

肯定会被骂娘的
分成10个还差不多
除非是refactor

看什么公司了，有的厂很夸张

去除unit test，超过50行我都想rubber stamp了

说实话有点多，按大厂普通标准来看。

核心business logic代码超过300行都很花时间review。

建议尽量break down成小cr。

1200 + 400?

嗯，我会直接留comment让你分成几个。除非是自动生成的文件也算在内

我直接就是个LGTM