gerrit提交代码评审,没有通过怎么办?

Posted Simon格子

tags:

篇首语:本文由小常识网(cha138.com)小编为大家整理,主要介绍了gerrit提交代码评审,没有通过怎么办?相关的知识,希望对你有一定的参考价值。

 

gerrit在提交代码后在线评审时,一般做法:

  • 方法一:如果问题较严重,则不通过,直接abandon,开发本地代码回退,重新生产commit提交评审,操作比较复杂;

  • 方法二:如果问题较轻微,则先通过,后面开发再提交一次commit修复上面的问题。

 

方法一可能需要开发熟练掌握本地commit回退的技术,重新提交commit,编写message存在重复劳动,比较低效;

方法二会导致有一些轻微的代码合入开发流,如果开发忘记修复这个问题,将遗留隐患;

如果开发事后修复了这个问题,同一个功能的内容出现了两次commit msg日志,会导致日志不美观。

 

方法三的做法如二:

第一步,下载钩子

从gerrit下载代码的时候,选择 【Clone with commit-msg hook】

 

如果你已经下载代码,但是没有下载commit-msg hook,那么你可以打开git bash,只执行 【&&】 后面的内容,其中【your-project-path】可以是绝对路径

scp -p -P 29418 xxx@gerrit.zte.com.cn:hooks/commit-msg  your-project-path/.git/hooks/

第二步,提交代码

正常提交,在git bash中依次执行命令,提交评审。

git add 文件
git commit -m ""
git push origin HEAD:refs/for/develop%r=代码评审人

第三步,代码评审

代码有问题,评审直接不通过(-2)

第四步,修改评审中的问题,再次提交

如果在上次提交后,未做git pull,也没有修改本地代码。(如果不是,请查看文章最后面的参考

可以直接修改评审中的问题,在git bash中依次执行命令,提交评审。

git add
git commit --amend --no-edit
git push origin HEAD:refs/for/develop

执行git push后,会看到[updated change],而不是new change。

此时,gerrit中待评审的评审单还是这个,但是代码内容已经变更。可以再次评审。 

以上是关于gerrit提交代码评审,没有通过怎么办?的主要内容,如果未能解决你的问题,请参考以下文章

git & gerrit & shell

如何在gerrit中修改已提交的change

ubuntu搭建gerrit+gitweb代码审核系统

gerrit 为每个工程设置提交的reviewer

软件项目简明代码评审流程

有没有更好的方法来调整gerrit插入代码大小基于规则和排除文件路径?