06 | 代码管理:如何使用 Git 与 GitHub 统一代码管理流程?

前言

本文来自拉勾网课程整理

在软件开发当中,代码管理一直是其中重要的一环,每当软件出现问题,我们就需要查看源码,及时发现其中的漏洞加以修复。并且,由于分工不同,软件开发需要多个人共同完成,如何保证每个人编写的代码符合要求,能够相互配合,也是一个重要的问题。所以,一个非常实用的代码管理工具和一套统一的代码管理流程在开发当中必不可少。

而在这方面,StackOverflow 曾调查发现,有超过 87% 的开发者使用 Git,有超过 82% 的开发者使用 GitHub 来进行代码托管和开发协作。可以说,熟练使用 GitGitHub 已经成为开发者的基本技能,而如何结合它们来规范代码管理也是 iOS 开发工程化实践当中的基础。

Git 分支管理

现代的软件开发活动通常需要多人参与。为了保证不同开发者可以同时贡献到同一个代码库,Git 提供了分支(Branch)来支持并行开发。不同团队有不同的 Git 分支管理方式,根据我们团队多年的经验,经过不断的完善,最终形成出一套简单并十分有效的 Git 分支管理规范,你可以参考下。这套规范是怎样的呢?

具体来说,我把所有的 Git 分支分成类:主分支,功能分支和发布分支,让它们各自承担不同的功能。

其中主分支也称为master或者main分支, 是Git代码仓库的默认分支。在Xcode 12以后,新建项目时也会默认生成命名为main的主分支。

主分支在软件开发中非常重要,它是我们App的唯一的信息源(Single source of truth),不论是编译出不同版本的App还是排查问题,都需要用到主分支的代码。并且,团队同事的代码,最终也必须汇总到这个主分支中,且不能出错。所以,所谓的统一代码管理流程,就是制定其他分支的代码如何合并到主分支的流程。

功能分支是我们在开发过程中建立的临时分支,它可以用来保存一次开发活动的状态。根据不同的开发活动,我把功能分支分成几个小类。

  • feature分支,当开发一个新功能的时候,我会为每一个功能建立一个叫作feature分支,当整个功能完成后就可以合并到主分支里面,并把该分支从 Git 代码仓库中删除掉。

  • bugfix分支,当发现Bug的时候,会专门建立一个bugfix分支,修改 Bug 后把它合并到主分支里面去。

  • spike分支,当我们探索或研究一些新技术(如 App Clips 功能)的时候,会建立一个叫作spike的分支。在得出结论以后,我才决定是否把该分支合并到主分支。如果探索失败,我就不会把相关的spike分支合并到主分支了。

在新建一个功能分支的时候,我们都遵循一定的命名规范,一般会把功能的描述作为分支的名称。例如当我们要开发一个点赞功能时,会把该分支可以命名为feature/add-like-button-to-moments-screen。又例如当我们需要修改用户头像的一个UI Bug时,会把分支命名为bugfix/fix-avatar-ui-bug

有那么多功能分支,在并行开发过程中我们该如何管理它们呢?

来看下面这幅。

761b2b7dca8504708cc58b55e919951d

举例来说,我有一个同事要开发点赞功能,他就从主分支最新的MC1commit 签出(checkout)并新建feature/add-like-button-to-moments-screen来进行点赞功能的开发。与此同时,另外一个同事发现了一个用户头像的UI Bug,她也从MC1commit 签出并新建了bugfix/fix-avatar-ui-bug来修改Bug

当头像的 Bug 修改完成后,开发组长把BC2commit 合并到主分支里面。在此之后点赞功能也开发完毕,开发组长又把该功能分支上的FC4commit 合并到主分支里面。这两个分支彼此独立,而且互不影响。

除了主分支功能分支以外,我们在发布App 的时候还使用到发布分支。

发布分支一般命名为release。每次当我们发布App之前都会把主分支的最新代码合并到发布分支去。因此发布分支会一直保存 App 发布版的源码记录。

有了发布分支以后,一旦发生严重的线上事故,例如出现引起高崩溃率的Bug时,我们可以马上在发布分支上进行修复。一般的做法是从发布分支上签出一个功能分支,例如当修复点赞按钮引起的崩溃时,我们可以建立一个叫作bugfix/fix-like-button-crash的功能分支,在修复该崩溃以后马上合并到发布分支,并提交到 App Store ,更新线上的 App

905804e2b318bc6ab5e8c27b682e0344

由上图可见,当我们把主分支最新的MC1commit 合并到release分支以后,提交了一个版本号为V2.0AppApp Store。当我们通过查看崩溃报告,得知新上线的点缀功能引起很多崩溃时,可以采取如下措施:

马上从release分支的V2.0commit 签出bugfix/fix-like-button-crash分支并修复该崩溃;

BC2commit合并回release分支,并立刻提交版本号为V2.1AppApp Store

把包含了该修复的V2.1commit 合并到main分支,保证主分支统一管理所有代码的更新状态。

有了上面讲的三大分支,以及它们签出合并的流程,我们就定义了一部分的代码管理规范——知道什么时候要使用哪个分支进行开发或者发布。但是你可能已经察觉到,当我们把功能分支合并到主分支的时候并没有进行任何的审查,万一有人不小心把Bug合并到主分支怎么办?

我们在这方面就吃过很多大的亏,有人把没有经过验证的代码直接 Push 到主分支,使得 App 的崩溃率提升了 10% 以上。为了保证主分支受控,我们就需要引入一个代码管理流程来管控主分支的合并过程。目前管控这一流程最有效的办法是使用Pull Request,众多代码托管和协作平台(如 GitLab,BitBucket 等)都支持 Pull Request 功能。下面我就以 GitHub Pull Request 为例子来看看它是如何工作的。

GitHub Pull Request 流程

Pull Request我把它简称为PRPR是一种团队协助的机制,在 GitLab 也叫作 Merge Request。当一个开发者完成一个功能的开发时,可以通过PR来通知团队其他成员进行代码审查和讨论。在协商并得到共识后可以通过PR把功能代码合并到主分支中。

一套完善的PR流程能有效降低沟通成本,提高代码质量,以及提升项目的自动化和工程化程度。

下面我们一起看一套完整的,并经过我们多年实践证明过的PR流程。

da373262269b24466982a720ba7604fc

这套流程分成步。

第一步,当我们要开发一个新功能或者修改一个 Bug 时,从主分支的签出并建立一个功能分支。这里需要注意的是,为了减少合并时出现的冲突,我们需要从主分支最新的一个 commit 签出。

第二步,我们可以在功能分支上持续开发并多次提交commit。因为我们是在独立的功能分支上进行开发,所有的变动都不会影响到主分支,所以可以放心修改所需的代码。

第三步,当我们完成一个功能的开发以后,就可以提交一个 PR了。为了避免合并冲突,我建议在提交PR前先 rebase 主分支的Git 历史。同时为了方便其他成员审查代码和参与讨论,我们在提交PR的时候需要清楚地描述所完成的功能,并把注意 事项,UI前后变动的区别,测试步骤等等一同写到PR描述文档里面。
在我们Moments App 项目中,为了方便开发者编写PR描述文档,我们建立了一个模板文件pull_request_template.md。当我们提交PR的时候,GitHub 会自动读取并准备好描述文档的模板,我们只需要填写相关内容即可。

2229d38beef20af8de989894169814a4

你可以到代码仓库查看该模板文件。

在提交PR的时候,我们还可以加上代码审查人(Reviewer)来通知他/她审查代码。同时也可以加上分类的标签(Label)来方便管理所有的PR,例如使用enhancement表示功能开发,使用bug表示Bug 修改。如果你的项目由多团队同时开发,我们还会为每个产品团队都建立一个标签,这样就能清楚地知道这个PR来自那个产品团队了。

f871e2bdd698e874f00548c6abd14739

第四步,一旦 PR 提交以后,其他成员就会收到通知消息,他们可以进行代码审查,并把反馈意见留言到 RP 里面。提交者可以根据留言来修改代码和提交新的 commit。当所有留言都修正和完善以后,可以再次通知审查人进行进一步的审查。

在项目之初,这个过程可能需要来回好几遍。但随着团队的不断磨合,这个过程会越来越快。因为代码审查人一般由项目中最资深开发者组成,他们的留言能帮助团队新成员迅速熟悉项目相关的技术和背景知识。同时能保证代码风格的一致性和提高整个项目的代码质量。

根据我们的经验,在代码审查上的投资所得到的好处,远高于所花的时间成本,我建议你或者你所在的团队在项目初期多投入PR审查工作。

第五步,大家通过协商得到共识以后,审查人就可以批准(approve)PR。这表示该PR可以随时合并到主分支了。

第六步,当PR得到批准以后,提交人可以把PR合并到主分支里面去。我建议在合并之前先rebase 主分支上的最新 commit,这样能保证合并过程没有冲突,并使得 Git的历史更加简洁。

上面就是我们的统一的代码管理流程。为了进一步保证少犯错误,我们可以通过修改GitHub上的配置来为每一步操作做自动检查,以保证所有成员都必须严格遵循一致的代码管理流程。

GitHub 配置

首先,我们可以把主分支保护起来,不允许任何人直接Push到主分支。

a4ab832b0ad64032294535203c9b38fe

然后要求所有 PR在合并之前都必须经过一个或以上的代码审查人批准。审查人的数量可以根据团队的情况进行调整。

6a2c70ae5385df2133d7f4ec616d3621

Github 检查到有某些条件不完全符合时,就不允许我们合并该 PR

44c12bf8d5181e89f6bd847c7eb0d717

比如,因为我们的 Moments项目配置了所有的 PR 都必须有一个或以上的代码审查人批准后才能合并。上图可以见GitHub PR页面上的Merge pull request按钮是失效的,并提示“Merging can be performed automatically with 1 approving review” (需要一个代码审查通过后才能合并)。

总结

在这一章介绍了Git的分支管理和 GitHubPull Request 流程。根据多年的项目经验,我给出了一套完整的统一代码管理流程,其重点是把 Git 分支分成三类,主分支、功能分支和发布分支,然后严格按照 GitHub Pull Request 流程来把代码合并到主分支里面。

c5b911d624fd3854236a6554cc98455b

有了这个规范,开发者就能严格遵循这个流程贡献代码,从而保证主分支在管控状态,同时也为项目的自动化和工程化打下基础。

源码地址:

pull_request_template.md
https://github.com/lagoueduCol/iOS-linyongjian/blob/main/.github/pull_request_template.md

你可能感兴趣的:(06 | 代码管理:如何使用 Git 与 GitHub 统一代码管理流程?)