首页   注册   登录
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX  ›  程序员

遇到沙雕同事怎么办

  •  
  •   jakezh · 74 天前 · 4731 次点击
    这是一个创建于 74 天前的主题,其中的信息可能已经有所发展或是发生改变。

    一个中东的 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。。。。。。看来这破公司是没啥前途 准备滚了

    30 回复  |  直到 2019-09-25 11:09:16 +08:00
        1
    nuistzhou   74 天前 via iPhone
    你在 issues 里面 @一下 manager 呗,起码以后说起来有凭有据
        2
    lbfeng   74 天前
    伊朗?
        3
    ericgui   74 天前
    啥公司?让兄弟们也避免采坑
        4
    andrewpsy   74 天前   ♥ 1
    不要说早点上线打他脸,如果有证据是你没卡住让垃圾上线了,你的脸也不保。
    这种事情越早 escalate 到上级越好。
        5
    fuermosi777   74 天前
    没看太懂,他啥意思?不想解决你提出的 review ?
        6
    dangyuluo   74 天前
    4 楼说的对,这种事不要自己解决,直接 explicitly 捅到上面,留下书面证据。

    另外让他 git signoff 一下,然后你留言说明问题。

    这种事情在国内建筑行业见得多了。有监理去施工单位检查的时候,施工方一个劲给好处要监理签字,这时候就考验你推卸责任的能力了。
        7
    terence4444   74 天前 via iPhone
    你有权不 approve,manager 同意了让 manager approve 去。
        8
    whoamiatvip   74 天前
    100 行 15 个 issue,可以。
    这事不用考虑,抓紧上报。
        9
    k9982874   74 天前 via iPhone
    你也可以往上捅啊,manager merge 了,你就发信给 manager 说明情况,随便 cc cto,留下书面证据以后撕逼用
        10
    cmdOptionKana   74 天前
    你都准备离开公司了,就不要怕事情闹大了
        11
    jakezh   74 天前
        12
    missmimimi   74 天前
    虽然没看太懂 但是评论看懂了 hhh
        13
    justrand   74 天前
    为什么有种似曾相识的感觉?在别的平台发过么?
        14
    nvioue   74 天前 via Android
    啥 楼主在谷歌上班???大佬好!!!
        15
    hereIsChen   74 天前
    100 行 15 个
    也真的太牛了
        16
    notreami   74 天前
    大家不能盯着 100 行 15 个,这种数字。
    15 个 issue 是什么?要是变量名规范、换行规范啥的、日志规范啥的,怕不是真有让人觉得你在刁难我胖虎
        17
    fvckDaybyte2   74 天前 via iPhone
    同意啥了……不 building the next generation of google services?
        18
    bccoder   74 天前 via Android
    大佬大佬
        19
    Hanggi   74 天前
    你们用啥 review 啊?
        20
    userdhf   74 天前
    I can live with how this is right now. 这句话该怎么理解?
        21
    deanguqiang   74 天前 via iPhone   ♥ 1
    @userdhf 大意是我们又不是造原子弹,现在这样凑合也没事
        22
    hhhsuan   74 天前
    楼主在国外?
        23
    pkookp8   74 天前 via Android
    @hereIsChen 15 个其实还好吧。就我而言,很多应该是
    单词换一个说法
    空格,换行
    日志级别
    注释
    换一种更通用的写法
    之类的
    真正的代码问题其实不多
        24
    wuweijia   74 天前
    @userdhf 就是差不多说我觉得没啥问题啊,可以合并。
        25
    userdhf   74 天前
    @deanguqiang
    @wuweijia
    谢谢大佬
    原来是句读
    I can live ( with ( how this is ) right now ).
        26
    zek   74 天前
    愣是没看懂楼主发的帖子
        27
    vcode   74 天前
    看不懂。。。
        28
    jakezh   73 天前
    @notreami #16
    大佬 这些问题有 lint 脚本自己跑, 不需要人工 review 的

    我挑的都是些逻辑 /err handling/锁 之类的问题
    15 个是总数,一共打回去三次 第四次胖虎不开心了
        29
    ericgui   73 天前
    @zek
    @vcode 我看懂了,这个在外企或者国外很普遍
        30
    SmiteChow   73 天前
    很正常 做好本职工作就是 我以前在一个公司被一个东欧程序员小窗说 don't want touch any code of yours

    当时他在做某个模块的重构,言下之意就是嫌弃我 code 垃圾呗 我只回了句 its your business,me too
    关于   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   2135 人在线   最高记录 5043   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.3 · 97ms · UTC 11:33 · PVG 19:33 · LAX 03:33 · JFK 06:33
    ♥ Do have faith in what you're doing.