如何在 `git` 介质中启动对旧代码的拉取请求？答案

【问题标题】：How to initiate a pull-request for an old piece of code in a `git` medium?如何在 `git` 介质中启动对旧代码的拉取请求？
【发布时间】：2019-09-02 13:20:12
【问题描述】：

我有一个经过一段时间开发的存储库。它包括一系列文件，我希望通过 Pull-request 获得同行评审。这个仓库中每个文件的一个拉取请求正是我所需要的。

过去，我曾针对新代码段或对现有代码的建议更改和修改发起代码审查。但是，我从来没有遇到过想要让存储库中已经存在的所有文件进行审查的情况。

我尝试的一种方法是从 master 分支中删除文件，并在我尝试从另一个分支（从 origin/master 分叉/克隆）再次推送它们时发出拉取请求。但是，我不确定这是否是最好的方法。

对现有代码进行代码审查的最佳方法是什么？

【问题讨论】：

您描述的过程（每个 PR 1 个文件）对我来说似乎不实用。它以明显未设计的方式使用 git，并且您的“拉取请求”中的“更改”根本不代表更改。但除此之外，您应该如何查看 lone 文件？代码库中的文件几乎总是相互关联的，以这种方式隔离代码审查几乎没有意义。仅查看 CSS 文件而不查看它适用的 HTML？我可能会错过图片中的某些内容，但我个人到目前为止还没有明白这一点。
@RomainValeri 逐个文件审查的原因是，在这个特定的 repo 中，每个文件都是 100% 独立于所有其他文件的。
是的，在这种情况下，它可能更有意义。

标签： git github pull-request github-codereviews

【解决方案1】：

对于现有代码，如“Recipe: code reviews for existing code with GitHub”所示，PR 将包含所有文件，而不是一次只包含一个文件。

例如：thibaudcolas/react-streamfield PR 1.

PR 到位后，您可以逐个保留 cmets 文件，如此评论中所示pullrequestreview。

见“Commenting on a pull request”

这可以为每个文件单独完成。

【讨论】：

当我将光标悬停在没有变化的行上时，我看不到这个“加号”。我有它的快照。但似乎无法在 cmets 中发布图像。
我发现我的光标可以在更改区域的上方和下方编辑 3 行。
@Rotail 文档提到“在存储库中打开拉取请求后，协作者或团队成员可以评论两个指定分支之间的文件比较，或者将项目中的一般 cmets 作为所有的。”确保您拥有正确的权利。并且您正在尝试比较视图（“文件已更改”）。