这是一个创建于 2214 天前的主题，其中的信息可能已经有所发展或是发生改变。
一个中东的 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