问题标签 [github-codereviews]

GitHub 提供了一个非常方便的 API（REST，v3）来查看存储库级别的所有内容。要查看一个人对代码审查的评论，我需要知道他/她审查了哪些存储库并爬取这些存储库中的所有 PR。但是，有时我不记得我进行代码审查的所有存储库。如果我需要找到一些特定的评论，那么记住它在哪个存储库中会很乏味。

那么，有没有更便捷的方法可以有效地搜索某人的 Code Review 评论呢？

我想使用 Sonarqube 来审查 LoadRunner/Jmeter 脚本以检查代码质量，在测试设计期间遵循标准和最佳实践。以前有人用过Sonarqube吗？

假设乔正在处理一项重大任务。他在一个包含 20 次提交的大型 PR 中提交了他工作的初步修订。现在其他开发人员花了很长时间审查该 PR 并要求进行一些更改/修复。Joe 现在用请求的更改修改他的提交，并（强制）推送 PR 的新版本。

现在审阅者需要从头开始重新审阅整个内容，即使新更改非常小。我还没有找到解决这个问题的方法。

过去，我使用过 Phabricator，它跟踪 PR 的每个修订版，以便您可以区分不同修订版之间的更改并精美地解决此问题（https://secure.phabricator.com/D13641?vs=32966&id =32967#toc）。例如，它允许您比较修订版 1 和修订版 2 的更改）。这样，您就可以只专注于审查少数新更改，而不是 20 次提交。

有没有办法我们可以用 Github 完成类似的事情？我想有一个我们不知道的不同工作流程。如果没有，人们有没有 github 的替代品？（除了Phabricator？）。

我能想到的唯一选择是不修改提交，而是创建新的提交而不是强制推送。这样做的问题是提交历史变得非常混乱，并且会有一些带有错误的提交，这些错误会在以后的提交中修复。

在下面附上 Phabricator 中的修订差异捕获，供那些不知道我在说什么的人使用。

提前感谢您的任何建议！

有一个相关的问题，允许评论更改集之外的部分代码。我只是想知道是否有人有巧妙的解决方法或实现类似的过程，也许更简单。

因此，我有一组不同的文件，这些文件随着时间的推移已经被处理和审查（如果相关，则在 monorepo 中）。在不同的点上，我们想让不同的人回顾整套。所以我们希望有人查看每个文件中的所有代码，整个文件，只是在整体安全/技术债务/api设计的上下文中进行审查。

就像 PR 一样，您需要与代码行关联的注释。我考虑将所有正在审查的文件移动到一个新目录中，但这不算作新内容，因为 git 更改检测算法。

有没有人有类似的过程或方法？

GitHub 文档状态：

打开拉取请求后，任何具有读取权限的人都可以查看和评论它提出的更改。

在公共 GitHub 存储库中，允许存储库所有者或协作者以外的人（即没有写入权限的人）审查拉取请求有什么意义？

我假设这可能是一个学习机会，但它似乎更有可能是浪费时间。具有写访问权限的人可以决定写入什么内容以及需要在代码中进行哪些更改才能实现。随机只读访问的开发人员不太可能确切地知道所有者想要什么。

此外，在这种情况下，对于拉取请求者和所有者来说，适当的礼仪是什么？

我们团队目前使用 gitlab 作为代码仓库。现在我们想集成一个代码审查工具，但我们希望代码还是直接提交给gitlab，而Phabricator只是用于代码审查。我能做些什么？有相关教程吗？

背景：
GitHub 属于客户端。我们团队中有一些新手，他们有时会错过基本的命名约定和其他编码协议。因此，如果任何高级人员想要在内部进行审查，除了创建 PR 之外别无他法。但是这个 PR 对客户也是可见的。

问题：我们是否可以使用一个工具（如果 GitHub 有一个功能更好）进行内部审查，然后创建一个干净漂亮的 PR？

我理解 PR 本身意味着它是出于审查目的，但至少在内部审查之后，它会有真正的问题需要调查，而不是一些琐碎/明显的问题。

我正在开发一个网络应用程序的大部分内容，并分配给我几张 jira 票来逐步完成它：例如

• 任务 1：创建显示项目列表的部分的主页
• 任务 2：创建一个包含详细项目描述的页面
• 任务 3：在详细页面添加创建/编辑/删除按钮以执行 CRUD 操作
• ...

因此，就 jira 票证而言，一切都很清楚，但是如果每个任务都依赖于前一个任务，我应该如何使用 github 拉取请求（PR）系统组织工作？

例如，我为任务 1 创建了一个新分支，完成了它并创建了一个 PR。然后开始处理任务 2：基于任务 1 分支创建了一个新分支，完成了工作，现在我必须创建另一个 PR，但是目标分支应该是什么？如果我选择 dev，PR 将包括 Task 1 的内容（这使得审查变得困难并且看起来不正确）。也许我应该等待第一个 PR 被合并，然后才开始处理任务 2（但是所有工作都会停止，直到任务 1 被合并）？

我遇到的另一个问题情况：假设在 sprint 管理结束时决定我们不再需要我们的应用程序中的列表页面（任务 1），详细页面就足够了（任务 2 及更高版本）。这两个 PR 仍在代码审查中，任务 2 基于任务 1 分支。在这种情况下我该怎么办？

我想查看 GitHub 上 git 存储库的单个分支中的所有代码。（我是教授，代码是学生论文的一部分。）任何一次提交都只会影响一小部分代码。我如何进行审查以允许我对来自任何提交的所有代码进行逐行评论？

我相信这个问题与如何进行 github PR 请求审查整个文件有关？，从未得到答复。

我正在使用codeowners文件要求 PR 批准，然后才能合并它们。我想做的是：

1. 要求所有 PR 至少获得一组代码所有者（工程师）的批准
2. 要求所有 PR 至少获得一组 QA 的批准

似乎代码所有者文件做了一个分层的事情，其中​​只有一个组拥有代码，您可以为某些目录创建规则，但这些只会覆盖默认代码所有者。

我当前的.github/CODEOWNERS文件如下所示：

有没有办法要求两个不同组的至少一个批准？