n********g 发帖数: 6504 | 1 今天早会,讨论项目包括如何review一老码工(60+)提交的修改。超过了2000+文件、
10万行的修改/升级。
我们这些破烂企业就是这么上production的。
上一个纪录是我创造的。全team一起review。谁不吭声就表示不反对。最后马内急Sign
off。
各位高大上企业是怎么Code Review这样的情况的。 |
l******t 发帖数: 55733 | |
n********g 发帖数: 6504 | 3 老先生这么有情怀。还开除?哪个企业不怕上劳工部黑名单:年龄歧视。
【在 l******t 的大作中提到】 : 可以都开除了
|
V********u 发帖数: 80 | |
V********u 发帖数: 80 | |
B*Q 发帖数: 25729 | |
r****1 发帖数: 1 | 7 银行里改一行都要层层审批
Sign
【在 n********g 的大作中提到】 : 今天早会,讨论项目包括如何review一老码工(60+)提交的修改。超过了2000+文件、 : 10万行的修改/升级。 : 我们这些破烂企业就是这么上production的。 : 上一个纪录是我创造的。全team一起review。谁不吭声就表示不反对。最后马内急Sign : off。 : 各位高大上企业是怎么Code Review这样的情况的。
|
n********g 发帖数: 6504 | 8 CR是制度。对股东和董事会负责。就是VP要checkin修改也要找人review,象征性提点
意见。
没人愿意review那就开大会集体review。
上次我的review开了几天会。这次怎么也得几天。年底了,正好干这个。
【在 B*Q 的大作中提到】 : 信任老元工 : 勿需CR
|
f**********u 发帖数: 348 | 9 廊庑也是人,也要生活的,哪能说开除就开除,小手。
上个帖子说廊庑网上讨薪,你有没有贪过你手下的五毛钱?
[在 lsunspot (小手) 的大作中提到:]
:可以都开除了 |
c********e 发帖数: 4283 | 10 包括若干一行上万字节的代码?
Sign
【在 n********g 的大作中提到】 : 今天早会,讨论项目包括如何review一老码工(60+)提交的修改。超过了2000+文件、 : 10万行的修改/升级。 : 我们这些破烂企业就是这么上production的。 : 上一个纪录是我创造的。全team一起review。谁不吭声就表示不反对。最后马内急Sign : off。 : 各位高大上企业是怎么Code Review这样的情况的。
|
|
|
d******d 发帖数: 6 | 11 很好,各人写的code就是饭碗的保障,不然那马公干不到60
【在 n********g 的大作中提到】 : CR是制度。对股东和董事会负责。就是VP要checkin修改也要找人review,象征性提点 : 意见。 : 没人愿意review那就开大会集体review。 : 上次我的review开了几天会。这次怎么也得几天。年底了,正好干这个。
|
n********g 发帖数: 6504 | 12 我还没看。应该不是。数字是GIT报告的。
如非必要,谁没事干改写。
【在 c********e 的大作中提到】 : 包括若干一行上万字节的代码? : : Sign
|
s*****l 发帖数: 7106 | 13 都是string replacement的话可以接受 |
d******w 发帖数: 2213 | 14 晕倒,怎么会发生这种事情?任何一个有经验的马工都应该知道如何break down
changes, 以可控的方式修改。 |
h*********n 发帖数: 11319 | 15 老马公有经验。自己拉的屎,出了问题你们所有人一起擦
Sign
【在 n********g 的大作中提到】 : 今天早会,讨论项目包括如何review一老码工(60+)提交的修改。超过了2000+文件、 : 10万行的修改/升级。 : 我们这些破烂企业就是这么上production的。 : 上一个纪录是我创造的。全team一起review。谁不吭声就表示不反对。最后马内急Sign : off。 : 各位高大上企业是怎么Code Review这样的情况的。
|
n********g 发帖数: 6504 | 16 请教一个涉及2000+文件、10万行的修改要break down到多少合适?
【在 d******w 的大作中提到】 : 晕倒,怎么会发生这种事情?任何一个有经验的马工都应该知道如何break down : changes, 以可控的方式修改。
|
n********g 发帖数: 6504 | 17 感觉像工会啊。有种一起炒掉。
【在 h*********n 的大作中提到】 : 老马公有经验。自己拉的屎,出了问题你们所有人一起擦 : : Sign
|
o****p 发帖数: 9785 | 18 这tm是kernel升级吧?有谁会一气提交这样的改动?
Sign
【在 n********g 的大作中提到】 : 今天早会,讨论项目包括如何review一老码工(60+)提交的修改。超过了2000+文件、 : 10万行的修改/升级。 : 我们这些破烂企业就是这么上production的。 : 上一个纪录是我创造的。全team一起review。谁不吭声就表示不反对。最后马内急Sign : off。 : 各位高大上企业是怎么Code Review这样的情况的。
|
d******w 发帖数: 2213 | 19 为啥会有这种事情呢?代码没有根据feature和component区分吗?如果有,这不就是一
个可以break down的地方吗? 如果你整个代码完全都在一起,2000+文件都是为了一个
东西,那就当我没说,一开始这么写的人就应该被打屁股。
【在 n********g 的大作中提到】 : 请教一个涉及2000+文件、10万行的修改要break down到多少合适?
|
c*********g 发帖数: 1 | 20 把所有變量名由駝峰改成了下劃線?
:为啥会有这种事情呢?代码没有根据feature和component区分吗?如果有,这不就是
一个可以break down的地方吗? 如果你整个代码完全都在一起,2000+文件都是为了一
个东西,那就当我没说,一开始这么写的人就应该被打屁股。
: |
|
|
l*******t 发帖数: 1430 | 21 所以你们现在理解牛和羊的工作水平了把。也是是叔为什么一上来就说全部开除
【在 d******w 的大作中提到】 : 为啥会有这种事情呢?代码没有根据feature和component区分吗?如果有,这不就是一 : 个可以break down的地方吗? 如果你整个代码完全都在一起,2000+文件都是为了一个 : 东西,那就当我没说,一开始这么写的人就应该被打屁股。
|
c*********g 发帖数: 1 | 22 廊五還是不要參與這種討論了,你也看不懂
:可以都开除了 |
d*******n 发帖数: 109 | 23 这种情况通常是 remove 了很多 unused code |