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

看是什么code 有些前端html或者config file或者 unit tests 动不动就是几百行 这种无所谓

匿名用户 发表于 2023-9-6 18:41
直接pip了

明白了，多谢多谢多谢

Chtholly 发表于 2023-9-6 18:52
150-150 rule: 指+/-diff都不超过150行。个人认为没必要非要按照150-150来发，但是它大体代表了一个对publi ...

非常感谢，下次就按这个量级来了。这次也是涉及了一些refactor，A写了好几个月的基础框架和一些功能后，去别的地儿了。然后来了一个B，B在没有理解好A代码的情况下，有点弄歪了，然后我接着B的歪的开始写，越写越发现和原有的codebase特别违和，不想歪下去了。

dayeye2006 发表于 2023-9-6 20:01
我直接就是个LGTM

职场老油条了属于

匿名用户 发表于 2023-9-6 19:11
说实话有点多，按大厂普通标准来看。

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

嗯好的，非常感谢指导。

领导安排的ticket里面包含了好几个 bullet points完成，原有代码也些问题。一般要求是一个PR对一个ticket。下次把ticket多break down一些

barkonstone 发表于 2023-9-6 21:02
看是什么code 有些前端html或者config file或者 unit tests 动不动就是几百行 这种无所谓

可以的，有config也有常量啥的，学到了！

匿名用户 发表于 2023-9-6 19:21
1200 + 400?

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

学到了，非常感谢！

看什么代码，如果是那种自动generate的xml/sql啥的，可能问题不大？

如果是logic code也太多了……为啥不多分成几个cr，这样你的contribution看起来也多不好吗？

这么大量的代码reviewer也要block出一长段时间去review，可能会不爽甚至拖延。

建议跟你tech lead/mentor聊聊该怎么break down这么大量的代码？？

还有一个小技巧就是尽量一个cr对应一个要修改的地方，不要一大片全改……比如你说有refactor，那你这部分完全可以拎出来单开一个cr（不过refactor别人的代码可能不太好直接这样发一个cr做，我的理解先找owner说一声比较好）。

除非是自动生成或者高度同质化（比如文字替换）的改动，否则一律拉去打靶