V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
git
Pro Git
Atlassian Git Tutorial
Pro Git 简体中文翻译
GitX
dingdong123
V2EX  ›  git

你们公司 git 提交代码流程是怎么样的?

  •  
  •   dingdong123 · 2020-01-17 14:49:58 +08:00 · 4019 次点击
    这是一个创建于 1532 天前的主题,其中的信息可能已经有所发展或是发生改变。

    最近入职一家新公司,然后这边提交代码每次提交都有人审核。比如我修改了某个位置,优化了执行的逻辑,或者注释掉了某段不需要的代码,审核人告诉我这样不行,需要还原,我当时很气,感觉这公司太僵化,甚至想马上离职。

    我这种想法对吗?还是很多公司都是这种流程

    22 条回复    2020-01-19 16:09:42 +08:00
    qq8758173803
        1
    qq8758173803  
       2020-01-18 08:43:51 +08:00
    如果薪水合适,这种流程便不算啥。我们小公司,开发就没那么多麻烦事了。
    guyeu
        2
    guyeu  
       2020-01-18 10:39:12 +08:00
    reviewer 告诉你不行肯定会告诉原因的,如果你觉得不合理可以刚啊。。
    lcsoft
        3
    lcsoft  
       2020-01-18 11:14:48 +08:00
    这流程看着没什么问题。
    orzorzorzorz
        4
    orzorzorzorz  
       2020-01-18 11:49:29 +08:00
    从字面看楼主表达的是,“我优化了代码,但老大不同意,很气,要离职”。
    从实际想,楼主没给出优化前后的 benchmark,看上去也没深入研究原先代码具体在其他什么地方、多少地方用到了,也没给老大说这么改不行的原因。直接一句“好气啊”,就开始顺理成章地想着,“啊,这公司不值得我努力,骑驴找马骑驴找马”,然后开始了摸鱼生活。
    我懂,我懂 :doge
    waterjiao
        5
    waterjiao  
       2020-01-18 12:39:16 +08:00
    code review 难道不会提出来吗?
    xxxy
        6
    xxxy  
       2020-01-18 14:21:02 +08:00
    这是正常流程,反而是让你随意改代码的公司不靠谱
    flyico
        7
    flyico  
       2020-01-18 16:48:38 +08:00
    一些大公司 PR 提出后修修改改,一周后才 merge,这都是常事
    zepc007
        8
    zepc007  
       2020-01-18 17:07:09 +08:00
    @flyico 这个狠😂😂😂😂
    wangyzj
        9
    wangyzj  
       2020-01-18 21:30:24 +08:00
    这个不是 git 流程的问题
    这个是管理流程的问题
    sungykk
        10
    sungykk  
       2020-01-19 00:31:47 +08:00
    这不很正常吗? 我们是要修改任何代码都需要 jira 上提 bug/feature id ---> git push 到 gerrit,组内至少一个人 review +1 , QA 测试 之后 verify , 总的代码管理人 review +2 之后才能 merge .
    liaojl
        11
    liaojl  
       2020-01-19 09:02:35 +08:00 via Android
    这才是正常流程吧
    cco
        12
    cco  
       2020-01-19 09:10:02 +08:00
    用优化过后的效果说服他不就行了:
    性能更好了?解耦了?可复用性强了?代码更短了?更健壮了?
    如果有,他不通过那奏是他的问题,否则。。
    Greendays
        13
    Greendays  
       2020-01-19 09:30:48 +08:00
    就算你的优化确实提高了性能,但是如果没有这个需求的话,我也不会不会让你通过的。这么努力干什么,下次有需求了再提交不好吗(
    wwwwzf
        14
    wwwwzf  
       2020-01-19 09:40:05 +08:00
    适应它。
    你就是英雄。
    xhf1024
        15
    xhf1024  
       2020-01-19 09:47:25 +08:00
    @orzorzorzorz 哈哈哈现在推理哈哈哈
    chitanda
        16
    chitanda  
       2020-01-19 10:36:18 +08:00
    我很羡慕这样的流程
    msg7086
        17
    msg7086  
       2020-01-19 10:37:39 +08:00
    你还是离职吧,找一家没有代码审核的公司,最好之前的程序员什么辣鸡代码都往库里灌,灌完就走人,让你接手来维护。
    Uyuhz
        18
    Uyuhz  
       2020-01-19 10:53:47 +08:00
    我很羡慕这样的流程+1。基本上一直单干,想找个人给我 code review 都没有
    atonku
        19
    atonku  
       2020-01-19 10:55:39 +08:00
    add->commit->push
    971586331
        20
    971586331  
       2020-01-19 15:22:54 +08:00
    有人看你的代码,这样编程水平才能提高啊,我们公司的代码都没人看,写了几年都是这个水平
    Tn5ohB1Yecdk3qCK
        21
    Tn5ohB1Yecdk3qCK  
       2020-01-19 15:27:00 +08:00
    这种流程很赞
    linZ
        22
    linZ  
       2020-01-19 16:09:42 +08:00
    这是在保护你, 保证你少返工,不知足么
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   3242 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 35ms · UTC 13:52 · PVG 21:52 · LAX 06:52 · JFK 09:52
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.