这是一个创建于 1886 天前的主题，其中的信息可能已经有所发展或是发生改变。
一个中东的 senior，写了个 100 行代码的程序让我 review 100 行里面找出来 15 个 issue
我说改完了我再 approve
这傻吊居然跟 manager 说
I don’t think we are building the next generation of google services… I can live with how this is right now.
然后手工做了个效果图，趁 manager 今天 WFH，偷偷跟 manager 汇报，manager 居然就 tmd 同意了。。。。。。。 我看到后想联系 manager 一直没联系上。。。。。。
懒得撕逼了，希望早点上线打他的脸。。。 这种人都能做 senior。。。。。。看来这破公司是没啥前途 准备滚了
 |
1
nuistzhou 2019-09-24 03:48:38 +08:00 via iPhone
你在 issues 里面 @一下 manager 呗，起码以后说起来有凭有据
|
 |
2
lbfeng 2019-09-24 05:13:26 +08:00
伊朗？
|
 |
3
ericgui 2019-09-24 05:53:17 +08:00
啥公司？让兄弟们也避免采坑
|
 |
4
andrewpsy 2019-09-24 06:16:38 +08:00  1
不要说早点上线打他脸，如果有证据是你没卡住让垃圾上线了，你的脸也不保。
这种事情越早 escalate 到上级越好。 |
 |
5
fuermosi777 2019-09-24 06:20:46 +08:00
没看太懂，他啥意思？不想解决你提出的 review ？
|
 |
6
dangyuluo 2019-09-24 06:40:42 +08:00
4 楼说的对，这种事不要自己解决，直接 explicitly 捅到上面，留下书面证据。
另外让他 git signoff 一下，然后你留言说明问题。 这种事情在国内建筑行业见得多了。有监理去施工单位检查的时候，施工方一个劲给好处要监理签字，这时候就考验你推卸责任的能力了。 |
 |
7
terence4444 2019-09-24 07:01:38 +08:00 via iPhone
你有权不 approve，manager 同意了让 manager approve 去。
|
 |
8
whoamiatvip 2019-09-24 07:51:32 +08:00
100 行 15 个 issue，可以。
这事不用考虑，抓紧上报。 |
 |
9
k9982874 2019-09-24 08:02:48 +08:00 via iPhone
你也可以往上捅啊，manager merge 了，你就发信给 manager 说明情况，随便 cc cto，留下书面证据以后撕逼用
|
 |
10
cmdOptionKana 2019-09-24 08:07:23 +08:00
你都准备离开公司了，就不要怕事情闹大了
|
 |
11
jakezh OP @nuistzhou
@lbfeng @ericgui @andrewpsy @fuermosi777 @dangyuluo @terence4444 @whoamiatvip @k9982874 @cmdOptionKana 感谢各位大佬 明天上班去撕逼 |
 |
12
missmimimi 2019-09-24 08:42:35 +08:00
虽然没看太懂 但是评论看懂了 hhh
|
 |
13
yEhwG10ZJa83067x 2019-09-24 08:43:46 +08:00
为什么有种似曾相识的感觉？在别的平台发过么？
|
 |
14
nvioue 2019-09-24 09:02:46 +08:00 via Android
啥 楼主在谷歌上班？？？大佬好！！！
|
 |
15
hereIsChen 2019-09-24 09:51:05 +08:00
100 行 15 个
也真的太牛了 |
 |
16
notreami 2019-09-24 11:18:25 +08:00
大家不能盯着 100 行 15 个，这种数字。
15 个 issue 是什么？要是变量名规范、换行规范啥的、日志规范啥的，怕不是真有让人觉得你在刁难我胖虎 |
 |
17
fvckDaybyte2 2019-09-24 11:23:25 +08:00 via iPhone
同意啥了……不 building the next generation of google services?
|
 |
18
bccoder 2019-09-24 12:21:47 +08:00 via Android
大佬大佬
|
 |
19
Hanggi 2019-09-24 13:44:29 +08:00
你们用啥 review 啊？
|
 |
20
userdhf 2019-09-24 14:00:23 +08:00
I can live with how this is right now. 这句话该怎么理解？
|
 |
21
deanguqiang 2019-09-24 14:06:29 +08:00 via iPhone 1
@userdhf 大意是我们又不是造原子弹，现在这样凑合也没事
|
 |
22
hhhsuan 2019-09-24 14:17:44 +08:00
楼主在国外？
|
 |
23
pkookp8 2019-09-24 14:18:17 +08:00 via Android
|
|
25
userdhf 2019-09-24 14:58:02 +08:00
|
 |
26
zek 2019-09-24 15:05:30 +08:00
愣是没看懂楼主发的帖子
|
 |
27
vcode 2019-09-24 15:11:13 +08:00
看不懂。。。
|
|
28
jakezh OP @notreami #16
大佬 这些问题有 lint 脚本自己跑， 不需要人工 review 的 我挑的都是些逻辑 /err handling/锁 之类的问题 15 个是总数，一共打回去三次 第四次胖虎不开心了 |
 |
30
SmiteChow 2019-09-25 11:09:16 +08:00
很正常 做好本职工作就是 我以前在一个公司被一个东欧程序员小窗说 don't want touch any code of yours
当时他在做某个模块的重构，言下之意就是嫌弃我 code 垃圾呗 我只回了句 its your business，me too |
Select Language