评审 Xcode 中的代码和协作

各位好 我是基伦 一名Xcode工程师 负责原始码控制技术 Xcode 13全面重新构想代码审核模式 并引进全新的协作作业流程 协助各位提升app的质量 在这节视频中 我会用Fruta这个app 详细示范典型的作业流程 我们会使用Xcode 13新的代码审核 以及拉拽请求功能 找出程序错误 取得反馈 整合更改的内容 并交付反馈给我的同事 现在开始吧 我的工作很大一部分是检视 用户回报的问题 为了处理程序错误 我必须先了解 问题发生的时间和原因 只要实时回看更改的代码 我通常就能用代码审核模式 找出原因 我们获得的 其中一个App Store评论 提到我们的app 最近更改了一些界面 看起来不太对 我对问题出在哪里心中有数 所以我会开始更改一些地方 我一边打字时 源码编辑器的编辑栏会显现变化 可以得知原先的数值真的很有用 我便能对更改的所有内容心知肚明 我可以跳入代码审核模式 只要点击编辑栏的代码审核钮 就能看到本地编辑 与最新修订版本的比较

在我处理文件的过程中 代码审核模式 会持续标出我编辑的内容 所以我总是能知道最后一次提交后 更改了哪些内容

对一个档案进行多次编辑后 底栏的步进器控制 会显示我已经更改了多少地方 点击箭号即可切换我更改的代码 如此一来 我不需要浏览整份档案 自己找出编辑过的地方 有时候 并列检视更改内容比较轻松 但是我比较喜欢 单行检视我的更改内容 Xcode可用这两种方式 显示更改的内容 通过右上角的编辑器菜单 我能切换这两种方式 我想确保我有找出 界面程序错误的所有问题 为了确切深入检查 并了解程序错误源自何处 我需要检视不同版本app的差异 我可以用底栏的提交控制 设定版本比较 Xcode 13让我可以在不同分支 标签和最新位置之间选择提交内容 所以我很快能看出任两个时间点 app的变化 我通常会在每次提交至 TestFlight或App Store时 切割一个产品分支 所以我会看一下与该产品版本 相关的两个代码版本

选择提交内容后Xcode会显示出 我的文件的两个历史版本 标示颜色会变成紫色和橘色 对应底栏显示的提交内容 跟前面一样 我可以用步进器控制 在两个版本间切换

我可以用单行或并列的方式看到比较 浏览完比较内容之后 我可以用底栏的重设钮 快速跳回最新的更改处 通常我对app的变更 会横跨多个档案 虽然在单一档案中 浏览更改处真的很有用 但我经常想要综观 我在整个项目中更改的所有档案 感谢原始码控制导航器的 新更改内容分页 我可以看到上次提交后 更改过的所有档案清单 点击任一个档案 会自动进入代码审核模式 并显现我对该档案的所有更改 我喜欢切换不同档案 快速提醒自己我编辑过的其他内容 既然我知道我的app 在展示食材时为什么会有问题 我可以开始新建新的分支 更改内容解决问题 我经常新建和切换分支 Xcode 13有很棒的新方法可以做到 Xcode工具栏 现在显示的是我目前的分支 有助于提醒我最近的工作项目 点击之后 可以在我最近正在处理的分支间 快速切换 在这个例子中 我会使用第二菜单 创建新的分支来处理问题

在代码审核模式浏览历史后 我能想到办法解决程序错误 既然我已经有建议的解决方法 我想知道更改代码的反馈 但是我不完全熟悉代码库这个部分 我希望我的组员可以确认 我更改的地方不会引发其他问题 为了跟同事协作变更代码 我要创建一个拉拽请求 通过Xcode 13 新整合支持GitHub和 Bitbucket Server的功能 我可以直接从Xcode 13创建 我可以从原始码控制弹出提示框 直接创建我的拉拽请求 Xcode会新建草拟的拉拽请求 把我的分支合并 进入默认的上游分支 在这个例子中是最重要的 我可以在发布之前 用解决问题获得的相关信息 准备我的拉拽请求 我提交和推送至分支的更动内容 会出现在更改导航器 拉拽请求的栏位 我就能预览发布后 会囊括的更改内容 本地更改栏位可以让我看到 在我提交和推送之前 不会包含在拉拽请求的档案 对拉拽请求内容满意的话 我就能加上标题和说明

我想特别请几位组员 审核这个更改版本 所以我点击参与者按钮 并标注他们为审核者

在我等待反馈并切换到其他分支时 Xcode推算出要展示的拉拽请求 Xcode会根据其来源码分支 以及你目前的分支 决定相关的拉拽请求 切换分支时 Xcode会自动找到已存的拉拽请求 稍早之前我已经发布我的拉拽请求 我的团队已提出一些反馈 现在来看一下反馈内容 在更改导航器选定拉拽请求后 能看到我的拉拽请求总览 以及发布之后的所有动态 团队对我的代码留下一些反馈 想要的话可以回复这些讨论 或点击“View Changes” 直接跳入代码审核模式中的档案

麦特留给我一些很好的建议 现在我们有机会 可以讨论我在Xcode的拉拽请求 并把它直接连结到原始码 我开始处理 并依据反馈内容 更新我的拉拽请求 现在 看一下组员留给我的评语 看起来他们支持我最新的更改版本 我明白同事认可这个拉拽请求 现在我只需要确认 我们持续整合作业流程的状态 由于我有使用Xcode Cloud 所以我也能在拉拽请求的 持续整合弹出提示框里 看到持续整合作业流程的状态 我可以很快看到检测结果的摘要 点击条目后 会直接跳到 详细的Xcode Cloud报告 想要更了解Xcode Cloud 以及在持续整合运行你的代码 请参考 “与Xcode Cloud有约”视频 好的 由于我的组员全都认可我的更改 我所有Xcode Cloud工作 都成功了 我准备好合并所有的拉拽请求 我会使用Pull Request动作 并选择“Merge” Xcode让我选择合并策略 可以决定要怎么把更改内容 整合进目标分支 我会选择默认 这包含Merge Commit 你也能输入非强制的提交信息 可是这个部分我不动它 一切看起来没问题 我就合并了

合并了我的拉拽请求之后 是时候切回其他作业 就像我觉得请求同侪反馈 很有帮助 他们也会请我审核他们的更改

除了我最近处理过的分支 原始码控制弹出提示框也会显示 跟我有关的拉拽请求 我可以看到我创建的拉拽请求 但是Xcode也会显示出 组员请我检视的所有拉拽请求 在原始码控制弹出提示框中 看起来有一位组员 正好请我提供反馈 现在去看看他们的分支并审核吧 跟我自己的拉拽请求一样 我也能看到他的总览、审核状态 以及我被标注之后 我组员的拉拽请求讨论串 现在来看一下他们的更改内容 我会从拉拽请求节点之下的 更改导航器档案进去

点击任一档案 都能让我进入代码审核模式 可以看到拉拽请求合并后 会套用的更改内容 再点击一次 我就能在编辑器里留评语给他 并在该行插入新的评语 我想我发现他的代码还有可改善之处 所以我留言给我的组员

我的反馈不多 这个拉拽请求看来已经没问题了 我会用右上角的这个动作认可

Xcode 13提供了审核代码 很棒的新方法 并能跟同事协作 代码审核模式是强大的工具 通过深入洞察 app原始码随时间的更改 可以调查app的问题 和更改的内容 代码审核模式也能提供新的 单行呈现模式 并扩充比较选项 让你的作业流程更有弹性 拉拽请求可让你与团队协作 在合并成主要分支之前 针对团队更改内容 给予和接收代码反馈 提升app的代码质量 讨论串就直接显示在app原始码中 让团队合作真正成为开发过程的一环 感谢收看 祝各位有美好的WWDC体验 [打击的音乐]