V2EX = way to explore
V2EX 是一个关于分享和探索的地方
Sign Up Now
For Existing Member  Sign In
• 请不要在回答技术问题时复制粘贴 AI 生成的内容
UserNameisNull
V2EX  ›  程序员

有用过 Gerrit 的吗?说出你们的评价吧

  •  
  •   UserNameisNull · Mar 25, 2019 · 3865 views
    This topic created in 2603 days ago, the information mentioned may be changed or developed.

    公司坚持使用 Gerrit,并执行 code review,不经过 reivew 代码无法发布测试。项目 owner 没空 review 时候很耽误时间。感觉还是 Gitlab 这是 protect branch 比较合理,merge 时候 review

    8 replies    2019-03-25 17:29:43 +08:00
    4BVL25L90W260T9U
        1
    4BVL25L90W260T9U  
       Mar 25, 2019
    gerrit 挺好用的呀?有啥不好的?
    mrgeneral
        2
    mrgeneral  
       Mar 25, 2019
    这个和 Gerrit 没有关系吧,就是 CR 这步在哪个阶段做的问题。

    Gerrit 最可怕的是不知道哪里冲突了,死活提交不上去,只能 git stash 然后重来😂
    mantis
        3
    mantis  
       Mar 25, 2019
    界面不太跟得上时代
    shily
        4
    shily  
       Mar 25, 2019
    “ owner 没空 review 时候很耽误时间”, 换成啥玩意都解决不了的问题。

    @mrgeneral 看提示!看提示!!看提示!!!重要的事情说三遍。目前还没有发现啥大问题,都是可以通过提示可以解决的。但我的很多同事都直接默默的解决或直接来问我。
    kaneg
        5
    kaneg  
       Mar 25, 2019
    是否有空 Review 是公司流程和开发人员整体水平的问题。
    Gerrit 能够做到每个人的提交不会影响到其他人,这个对一个大 team 开发是很有好处的。
    至于没有 reivew 的 代码无法发布测试完全是你们的 DevOps 的水平来决定的。我们这边完全可以做到自己的代码提交到 Gerrit 之后可以 构建-> 测试->代码质量扫描( Sonar )-> 发布个人 build-> 安装个人实例一条龙服务。而这一切与 Code Review 都是并行的。也就是做到开发提交一行代码到 Gerrit,等待若干时间就可以在一个全新创建的个人实例上看到自己的改动是否与期望一致。而是否能够通过 Code Review 则是 Reviewer 的事。

    当然 Gitlab 能否做到这一切我不清楚,毕竟没用过。但是 Gerrit 带来的好处却是实实在在的。
    SevenJ
        6
    SevenJ  
       Mar 25, 2019
    用过,Gerrit 没毛病,还是人的问题
    mrgeneral
        7
    mrgeneral  
       Mar 25, 2019
    @shily 我记得很多次提示 changeId 缺失,然后其实不是本次提交带来的,可能是 merge 了老的 commit 带来的,老的 commit 没有 changeId,所以需要回到老的 commit 去补上 changeId,又会带来新的连锁反应,so,最后还不如重新拉个分支,把自己的改动重新提上去。

    PS:我们是中途接入的 Gerrit。
    idamien
        8
    idamien  
       Mar 25, 2019
    再一个主要是看到底是在审查什么,如果单纯的用 gerrit 的目的是为了格式,代码规范什么的,我觉得一点也没有必要。

    做代码审查的人,必须要对代码足够熟悉,否则你提交了修改,他也是看不懂深层次的东西,也是白搭。
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Solana   ·   6171 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 47ms · UTC 02:43 · PVG 10:43 · LAX 19:43 · JFK 22:43
    ♥ Do have faith in what you're doing.