练习 - 创建、评审和合并拉取请求

  • 5 分钟

你已完成将队列添加到网站的工作。 现在，网站开发团队已准备好将更改合并到主分支。 在本练习中，你将为更改创建和合并拉取请求。

在此过程中，你将：

  • 创建拉取请求。
  • 评审拉取请求。
  • 完成拉取请求。
  • 验证是否已合并更改。

创建拉取请求以合并功能分支

由于不能将更改直接推送到存储库的主分支，因此需要创建拉取请求。

  1. 在浏览器中，转到“代码”。

  2. 选择“2 个分支”以列出 GitHub 存储库中的分支。

    显示存储库页面的 GitHub 的屏幕截图，其中突出显示了分支列表的链接。

  3. 在“添加订单队列”旁边，选择“更多”图标 (...)，然后选择“新建拉取请求”。

    显示分支列表的 GitHub 的屏幕截图，其中突出显示了 add-orders-queue 分支的新拉取请求按钮。

  4. 创建拉取请求时，请注意 GitHub 已自动将 Git 提交消息用作拉取请求标题。

    将说明更新为以下文本：

    此 PR 添加一个新的 Azure 存储队列来处理订单，并更新网站配置以包含存储帐户和队列信息。

  5. 选择“创建拉取请求”。

    显示拉取请求创建页面的 GitHub 的屏幕截图，其中突出显示了用于创建拉取请求的按钮。

  1. 在浏览器中，转到“Repos”>“文件”。

    注意 Azure DevOps 显示了一个横幅，指示 add-orders-queue 分支中发生了更改。 横幅用于为这些更改创建拉取请求。

    显示存储库文件列表的 Azure DevOps 的屏幕截图，包括提供的用于创建拉取请求的横幅。

  2. 选择“创建拉取请求”。

  3. 在创建拉取请求的页面上，注意 Azure DevOps 已自动将 Git 提交消息用作拉取请求标题。

    将说明更新为以下文本：

    此 PR 添加一个新的 Azure 存储队列来处理订单，并更新网站配置以包含存储帐户和队列信息。

  4. 选择“创建”。

    显示拉取请求创建页面的 Azure DevOps 的屏幕截图，其中突出显示了用于创建拉取请求的按钮。

评审拉取请求

通常，拉取请求由除作者之外的其他人员评审。 在本例中，你将假装是另一个团队成员，并评审自己的拉取请求。

  1. 在“拉取请求”页上，选择“文件已更改”选项卡。

    GitHub 的屏幕截图，其中显示了拉取请求中更改文件的选项卡。

    GitHub 显示此拉取请求中已更改的文件。 注意它突出显示了所有已更改的行，让你可以轻松查看应评审的内容。

    提示

    假设你正在为自己的团队评审此内容。 是否需要提出任何建议？

  2. 在已更改的 main.bicep 文件中，将鼠标悬停在第 18 行上，然后选择带加号 (+) 的按钮。

    GitHub 的屏幕截图，其中显示了对 main.bicep 文件的更改。鼠标悬停在第 18 行，突出显示了用于添加注释的按钮。

  3. 在“评论”框中，输入以下文本：应采用大写形式吗？

  4. 选择“开始评审”。

    显示注释字段的 GitHub 的屏幕截图，其中突出显示了用于开始评审的按钮。

    提示

    GitHub 不允许用户批准自己的拉取请求。 在这里，你将评论拉取请求，但不会予以批准。 当你处理自己团队的拉取请求时，可以在这时批准拉取请求，表示你同意合并拉取请求。

  5. 选择“完成评审”。

  6. 在出现的评审面板中，选择“提交评审”。

    显示完成评审的面板的 GitHub 的屏幕截图，其中突出显示了用于提交评审的按钮。

    GitHub 会将你返回到拉取请求的“对话”选项卡。

  1. 在“拉取请求”页上，选择“文件”选项卡。

    Azure DevOps 的屏幕截图，其中显示了拉取请求中更改的文件。

    Azure DevOps 显示此拉取请求中已更改的文件。 注意它突出显示了所有已更改的行，让你可以轻松查看应评审的内容。

    提示

    假设你正在为自己的团队评审此内容。 是否需要提出任何建议？

  2. 在已更改的 main.bicep 文件中，将鼠标悬停在第 18 行上，然后选择评论按钮。

    Azure DevOps 的屏幕截图，其中显示了对 main.bicep 文件的更改。鼠标悬停在第 18 行，突出显示了添加注释的按钮。

  3. 在“评论”框中，输入以下文本：应采用大写形式吗？

  4. 选择“评论”。

    显示注释字段的 Azure DevOps 的屏幕截图，其中突出显示了“注释”按钮。

    浏览器窗口的宽度可能会影响注释对话框的显示效果。 注释将打开“讨论”对话框，而不是屏幕截图中所示的内联注释。

  5. 选择“批准”。

    Azure DevOps 的屏幕截图，其中显示了拉取请求的“批准”按钮。

    选择“批准”后，将“设置自动完成”更改为“完成”。 稍后在本单元将使用该功能。

响应拉取请求评审

在创建或评审拉取请求时，可以参与有关其内容的对话。 假设你是此文件的作者，你想要响应审阅者的评论。

  1. 使用以下评论响应拉取请求评审：否，存储队列必须采用小写名称。

  2. 选择“评论”，然后选择“解决对话”指示该行上的讨论结束。

    显示响应注释的 GitHub 的屏幕截图，其中突出显示了用于输入注释和解决对话的按钮。

  1. 在“拉取请求”页上，选择“概述”选项卡。

    显示“概述”选项卡的 Azure DevOps 的屏幕截图。

  2. 现在，假设你是此文件的作者。 使用以下评论响应拉取请求评审：否，存储队列必须采用小写名称。

  3. 选择“回复并解决”以指示进行中的讨论已结束。

    显示响应注释的 Azure DevOps 的屏幕截图，其中突出显示了用于回复和解析的按钮。

完成拉取请求

网站的开发团队已确认可以让你将订单发送到队列，因此你可随时完成并合并拉取请求。

拉取请求已获得批准。 网站的开发团队已确认可以让你将订单发送到队列，因此你可随时完成并合并拉取请求。

  1. 选择“合并拉取请求”。

    显示拉取请求的 GitHub 的屏幕截图，其中突出显示了用于合并的按钮。

  2. GitHub 要求你确认合并。 当 GitHub 合并拉取请求时，会创建提交并自动生成提交消息。 选择“确认合并”。

    显示拉取请求的 GitHub 的屏幕截图，其中突出显示了用于确认合并的按钮。

    你的拉取请求已合并，新功能现在位于存储库的主分支中。

  3. 完成你的功能分支后，最好将它们删除。 删除分支有助于避免将来团队成员对哪些工作仍在进行中感到困惑。 选择“删除分支”。

    显示拉取请求的 GitHub 的屏幕截图，其中突出显示了用于删除分支的按钮。

  1. 选择“Complete”（完成）。

    显示拉取请求的“完成”按钮的 Azure DevOps 的屏幕截图。

  2. 在“完成拉取请求”中，使用默认设置。 选择“完成合并”。

    显示拉取请求完成面板的 Azure DevOps 的屏幕截图，其中突出显示了用于完成合并的按钮。

    你的拉取请求已合并，新功能现在位于存储库的主分支中。

    当你合并拉取请求后，Azure DevOps 会自动删除功能分支。 完成你的功能分支后，最好将它们删除。 删除分支有助于避免将来团队成员对哪些工作仍在进行中感到困惑。

验证更改

合并拉取请求后，最好确认已成功合并更改。

  1. 转到“代码”。

  2. 转到 deploy/main.bicep 文件，然后转到 deploy/modules/appService.bicep 文件。

    GitHub 的屏幕截图，其中显示了合并拉取请求后的存储库文件列表。

    注意队列和其他更改现在位于这些文件中。

  1. 转到“Repos”>“文件”。

  2. 转到 deploy/main.bicep 文件，然后转到 deploy/modules/appService.bicep 文件。

    注意队列和其他更改现在位于这些文件中。