git merge注意事项

从使用git开始,遇到过各种各样的merge事故,本团队内的大大小小的事故就不下10起。小的merge错误,有的被测试当做bug提出,有的则是自己对该模块的代码再次修改时,才能再次发现;大的merge错误,则会block整个团队提交代码的动作,甚至会影响发版。本文主要讨论一些merge的注意事项以及小技巧。

1. 强烈建议使用Beyond Compare merge代码

Android Studio的VCS某些功能很好用,如批量pull与push,但是merge代码的功能实在不敢恭维。举一个比较简单的例子:

as-left-right.png

上图中,冲突的解决,需要手动选择local(left)与remote(right),常看到新手直接选择remote,或者直接选择local,把其中一个丢弃,甚至直接自信点击右下角apply,这无疑造成了一起merge事故。
来看看Beyond Compare的merge方式:
bc-left-right.png

图片下半部分中,Beyond Compare会给出建议的merge结果,可视化效果也做得很不错,让用户有意识去选择local(left),或者remote(right),处理过后的结果还不满意,也可以自己手动编辑。在这个例子中,可以「choose left then right」。
以上的例子还算简单,遇到复杂且数量多的冲突,Beyond Compare会明显比Android Studio好用,而且基本都会提供靠谱「建议结果」。需要注意的是,即使提供了「建议结果」,也需要一一review,不要想当然。另外,如果自己不了解业务层代码,或者实在不知道怎么解决一个冲突,请叫人叫人叫当事人

2. 尽量避免冲突

避免冲突,merge的时候就会省去很多力气。有几个避免冲突的小技巧(欢迎补充):

  • 公共模块,尽量不要在文件的末尾增加代码。如styles.xml这类的公共模块文件,如新增的style直接在末尾增加,其他开发者也把新增的style添加在末尾,merge时则会造成冲突。如果没有严格的顺序要求,可以把新增的style增加到一个随机的中间位置。
  • 禁止大面积format代码。有的同学有强迫症,写代码时容不下一两个空格的差异,然后顺手按下cmmd+shift+f,就把整个文件format了。merge的时候,发现几十个冲突(如果其他人也编辑了这个文件),用的还是Android Studio,瞬间懵逼。如果非要format代码,可以用鼠标选中看不顺眼那几行再format
    选中选中.png
  • 提前沟通,减小冲突的可能性。如果某些模块需要两个人一起修改,这时两人需要提前沟通各自需要修改的地方,merge代码时可能造成冲突的地方,然后再分工,尽量减小冲突,而且一定不要有重复工作。
3. 注意remote被删除了的情况

可能存在这种情况,merge的时候,remote上的a.java文件被删除了,而local又不小心地(1)在a.java增加了一些空格;(2)甚至进行了业务代码的修改。
首先,要识别出remote文件被删除并造成冲突的情况。如下图:

remote-delete-merge.png

图片中的①位置diff区域,可以看出是空白的,而执行②,使用Beyond Compare merge时,Beyond Compare会没有反应,且图中③的位置会莫名其妙多出好多merge时的tmp文件。这时就可断定出remote中的该文件被删除了
针对以上(1)的情况,可以选择「resolve using 'theirs'」直接把文件删除。而针对(2)的情况,需要与文件的删除者沟通,再把自己修改的代码挪到相应的地方。
反之,也可能出现local中文件被删除,remote中有人修改该文件的情况:
local-delete-test.png

可以看到diff区域全是增加(+)的绿色。处理的方法和上述类似。

4. 新建文件重构时,注意及时同步代码

如果要新建a.java与b.java文件,对c.java文件进行重构,2个星期后,重构完成,需要保证这两个星期内在c.java的修改,也同步到重构后的a.java与b.java中。

5. import相关的冲突基本可以忽略,使用BC推荐的解决冲突方式即可
merge-import.png

有时候import的冲突也比较多,解决起来也比较繁琐,其实可以跳过手工合并,直接选择Beyond Compare推荐的结果,即使推荐错了,编译期也能发现编译错误,到时候再自动import即可。

6. Merge时的commit message使用默认的

常常看到有人修改了merge节点的commit message,替换成俩字:merge。务必使用默认的commit message,如下图,带有conflict信息:


merge commit

保留的conflict信息也对日后有用。

总结,使用好的解决冲突工具:Beyond Compare,当然还有Source Tree这样优秀的git工具。尽量减小冲突。以认真的态度对待merge。

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 213,558评论 6 492
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 91,002评论 3 387
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 159,036评论 0 349
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 57,024评论 1 285
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 66,144评论 6 385
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 50,255评论 1 292
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 39,295评论 3 412
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 38,068评论 0 268
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,478评论 1 305
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 36,789评论 2 327
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 38,965评论 1 341
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 34,649评论 4 336
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 40,267评论 3 318
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 30,982评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,223评论 1 267
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 46,800评论 2 365
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 43,847评论 2 351

推荐阅读更多精彩内容

  • Spring Cloud为开发人员提供了快速构建分布式系统中一些常见模式的工具(例如配置管理,服务发现,断路器,智...
    卡卡罗2017阅读 134,644评论 18 139
  • 1. GIT命令 git init在本地新建一个repo,进入一个项目目录,执行git init,会初始化一个re...
    江边一蓑烟阅读 793评论 0 0
  • 热带雨林中的咖啡馆 项目简介 设计师:gregory melitonov, ines guzman 地址:危地马拉...
    图羚阅读 932评论 0 0
  • 金钢牛阅读 272评论 0 0