分页： 1 / 1
#1 这种情况将军们是怎么处理的？
发表于 ： 2025年 7月 31日 11:33
由 只儿豁阿歹
叔不是科班出身的马工。
最近公司突然推出新政，每个sprint员工必须提交一个以上pull request, 还必须一人以上review。
本来我们组的工作就是自娱自乐，谈不上什么“production"。
然后我就和本组的一个阿拉伯女人互加了reviewer, 因为我们俩干的活比较类似。
叔先交了两个pull request, 被阿拉伯人秒过了
今天她也给叔发了一个。书一看，写的跟屎一样，到处是错
叔要是approve, 这个玩意随便谁看都有问题
叔要是不approve, 这段sql 根本就不是用在production上，只有那个阿拉伯人自己会用
叔需要评论告知她吗？
#2 Re: 这种情况将军们是怎么处理的？
发表于 ： 2025年 7月 31日 12:09
由 赖美豪中
这又是会计出身cio想出来的? 你可以改完了以后发给他让她照着改了进repo
只儿豁阿歹 写了： 2025年 7月 31日 11:33
叔不是科班出身的马工。
最近公司突然推出新政，每个sprint员工必须提交一个以上pull request, 还必须一人以上review。
本来我们组的工作就是自娱自乐，谈不上什么“production"。
然后我就和本组的一个阿拉伯女人互加了reviewer, 因为我们俩干的活比较类似。
叔先交了两个pull request, 被阿拉伯人秒过了
今天她也给叔发了一个。书一看，写的跟屎一样，到处是错
叔要是approve, 这个玩意随便谁看都有问题
叔要是不approve, 这段sql 根本就不是用在production上，只有那个阿拉伯人自己会用
叔需要评论告知她吗？
#3 Re: 这种情况将军们是怎么处理的？
发表于 ： 2025年 7月 31日 12:23
由 netflix
只儿豁阿歹 写了： 2025年 7月 31日 11:33
叔不是科班出身的马工。
最近公司突然推出新政，每个sprint员工必须提交一个以上pull request, 还必须一人以上review。
本来我们组的工作就是自娱自乐，谈不上什么“production"。
然后我就和本组的一个阿拉伯女人互加了reviewer, 因为我们俩干的活比较类似。
叔先交了两个pull request, 被阿拉伯人秒过了
今天她也给叔发了一个。书一看，写的跟屎一样，到处是错
叔要是approve, 这个玩意随便谁看都有问题
叔要是不approve, 这段sql 根本就不是用在production上，只有那个阿拉伯人自己会用
叔需要评论告知她吗？
什么组竟然没有production也行？竟然没有被砍？
#4 Re: 这种情况将军们是怎么处理的？
发表于 ： 2025年 7月 31日 12:55
由 fantasist
这几年只听说过按代码行数计算performance的扯淡玩法。
一个sprint是两周？只用交一个PR就能通过的要求也是够低的，随便怎么样都能过。你们公司好不专业，code review听起来是optional的……
#5 Re: 这种情况将军们是怎么处理的？
发表于 ： 2025年 7月 31日 12:58
由 牛河梁
fantasist 写了： 2025年 7月 31日 12:55
这几年只听说过按代码行数计算performance的扯淡玩法。
一个sprint是两周？只用交一个PR就能通过的要求也是够低的，随便怎么样都能过。你们公司好不专业，code review听起来是optional的……
和卖牠没法比。很多小公司reviewer都凑不齐。
#6 Re: 这种情况将军们是怎么处理的？
发表于 ： 2025年 7月 31日 13:46
由 赖美豪中
我们组现在得code全部进不了production, 因为不符合三锅总设计师发明的code quality control. 什么test coverage, streamline unit testing. 这种sb玩意除了写网页得，99%得正经软件都实现不了啊
netflix 写了： 2025年 7月 31日 12:23
什么组竟然没有production也行？竟然没有被砍？
#7 Re: 这种情况将军们是怎么处理的？
发表于 ： 2025年 7月 31日 13:48
由 netflix
赖美豪中 写了： 2025年 7月 31日 13:46
我们组现在得code全部进不了production, 因为不符合三锅总设计师发明的code quality control. 什么test coverage, streamline unit testing. 这种sb玩意除了写网页得，99%得正经软件都实现不了啊
那你们组存在的价值是什么？既然不用你们的code，那把你们组原地解散也没什么影响
#8 Re: 这种情况将军们是怎么处理的？
发表于 ： 2025年 7月 31日 13:49
由 赖美豪中
代码行数只适用于java, c++马工，写这玩意得三锅每天再简历上贴代码行数
fantasist 写了： 2025年 7月 31日 12:55
这几年只听说过按代码行数计算performance的扯淡玩法。
一个sprint是两周？只用交一个PR就能通过的要求也是够低的，随便怎么样都能过。你们公司好不专业，code review听起来是optional的……
#9 Re: 这种情况将军们是怎么处理的？
发表于 ： 2025年 7月 31日 13:50
由 赖美豪中
user可以按test 用啊，我们整个产品都tmd是test group里的了，哈哈哈
netflix 写了： 2025年 7月 31日 13:48
那你们组存在的价值是什么？既然不用你们的code，那把你们组原地解散也没什么影响
#10 Re: 这种情况将军们是怎么处理的？
发表于 ： 2025年 7月 31日 14:03
由 牛河梁
netflix 写了： 2025年 7月 31日 13:48
那你们组存在的价值是什么？既然不用你们的code，那把你们组原地解散也没什么影响
老领导要小心，年底烙印会计CFO就可以发饭盒了
#11 Re: 这种情况将军们是怎么处理的？
发表于 ： 2025年 7月 31日 15:05
由 fantasist
牛河梁 写了： 2025年 7月 31日 12:58
和卖牠没法比。很多小公司reviewer都凑不齐。
主要由eng quality决定。再小的team只要有四五个人以上，就应该互相review了，不一定多严格但至少要看一眼。
#12 Re: 这种情况将军们是怎么处理的？
发表于 ： 2025年 7月 31日 15:07
由 只儿豁阿歹
赖美豪中 写了： 2025年 7月 31日 12:09
这又是会计出身cio想出来的? 你可以改完了以后发给他让她照着改了进repo
你猜中了，我们新来个烙印CTO
#13 Re: 这种情况将军们是怎么处理的？
发表于 ： 2025年 7月 31日 15:48
由 juderiverman
美华的生活经历真相似
#14 Re: 这种情况将军们是怎么处理的？
发表于 ： 2025年 7月 31日 19:15
由 agnis
约她一起改代码，多浪漫的事。活该被烙印欺负
只儿豁阿歹 写了： 2025年 7月 31日 11:33
叔不是科班出身的马工。
最近公司突然推出新政，每个sprint员工必须提交一个以上pull request, 还必须一人以上review。
本来我们组的工作就是自娱自乐，谈不上什么“production"。
然后我就和本组的一个阿拉伯女人互加了reviewer, 因为我们俩干的活比较类似。
叔先交了两个pull request, 被阿拉伯人秒过了
今天她也给叔发了一个。书一看，写的跟屎一样，到处是错
叔要是approve, 这个玩意随便谁看都有问题
叔要是不approve, 这段sql 根本就不是用在production上，只有那个阿拉伯人自己会用
叔需要评论告知她吗？
#15 Re: 这种情况将军们是怎么处理的？
发表于 ： 2025年 7月 31日 21:07
由 heteroclinic
我约莫是MS/GHB给信用卡返点，FBI也不管管么？
关FBI屁事。
书在这问题上吃了大亏。你把上面文件好好看看，最好的策略是随大流。飞机掉下来，跟你没有任何关系。
如果你被TARGETED，不要难过，赶紧发简历。已经内定了。
#16 Re: 这种情况将军们是怎么处理的？
发表于 ： 2025年 8月 2日 16:17
由 zmz123
这是啥组，赶紧找工作啊，内部转也行。
我要是在楼主位置，肯定不会马上 approve，至少加个 comments，但是用语要委婉，
