问题标签 [github-codereviews]

我应该忽略 GUI 类的自动代码审查错误吗？此类由 Windowbuilder 生成。我应该做出不同的做法，让它在自动代码审查中获得 A 级，我应该怎么做，或者我应该忽略它们，因为这种类型的类没有真正的复杂性？唯一描述窗口组件，仅此而已。代码审查告诉我：

• 分配分支条件
• 实例变量太多
• 功能太长
• 总复杂度太高

这是代码

在我们的 GitHub 存储库中，我们最近启用了在合并分支规则之前需要拉取请求审查。develop

此外，我们有一个小 bash 脚本，它创建一个执行以下操作的新版本：

• 在 pom.xml 中更新版本
• git 提交
• 创建新的 git 标签
• git 推送

我们在分支中运行这个脚本develop，它的提交有一定的名称模式：New version: ${new-version}.

问题

由于“在合并之前需要拉取请求审查”规则，“新版本”-提交现在被拒绝，因为它们没有接受，这似乎是完全合理的。

问题

有没有办法不对具有特定名称模式的提交强制执行“合并前需要拉取请求审查”规则？

当我开始代码审查时，我可以添加多个带有“待定”状态审查的评论，然后一次提交所有评论，而不是一个一个提交。

如果我关闭审查并返回它，那些待处理的评论将被保存，但它需要我返回相同的拉取请求。有没有办法专门搜索那些我已经开始审核但尚未提交的 PR？我尝试使用搜索，commenter:me is:open但这is:pending只会返回我已完全发表评论的 PR。

这是我的工作流程：

做一些改变

然后在代码审查中，我看到了一些空行，所以我没有逐行更改格式，而是将该文件重置为类似于 origin/master

那么这一次，我只添加了需要更改的部分，没有触及任何其他部分。

但在 github 中，我仍然看到一些不需要的行。我怎样才能删除它们？

例如，这些行完全相同，它们不应突出显示：

笔记：

• 我无法显示其他类似的代码（因为它是工作仓库）或有这样的空行。
• 没有 gitook 可以运行该格式。
• 这是一个詹金斯文件

作为团队成员，我们致力于使用 Github 创建一个 Android 应用程序。我们使用 PR 和代码审查。我们也使用 Android Studio 进行开发，

在没有实际看到布局的情况下很难对布局进行代码审查。我想知道是否有一些工具或方法可以共享应用程序实例以查看更改？

This is my protected branch (master) configuration:

And this is what I tried:

So I have a github my-repo repository. I have a build-pipeline GitHub Action which I have kept as required for a commit to be added to master. Here's what the required checks statement says:

Choose which status checks must pass before branches can be merged into a branch that matches this rule. When enabled, commits must first be pushed to another branch, then merged or pushed directly to a branch that matches this rule after status checks have passed.

So here's what I expected would happen. I can push from my local master to remote master (which is protected) directly because of that last statement^, but the commit will be in an intermediate state at remote because build-pipeline status check have not been successful yet. I am aware that that's not how git works. But then what's the point of that last statement^ when it is forcing me to create a non-protected branch and raise a pull request to perform build-pipeline status check and then merge to master.

我是公共 GitHub 存储库的所有者，但我无法添加任何人来审核我的拉取请求。它只显示“没什么可显示的”。我尝试了另一个类似问题的建议=向存储库添加协作者。但我无法添加自己，因为我是存储库的所有者。

我在 github 上查看了一个拉取请求并请求更改。他们做出并推动了改变。现在我找不到重新审核并说“批准”的按钮。它在哪里？

我还查看了文档。没运气。

我想把它减少到[request->headers().getMethodValue())].

我对 C++ 相当陌生。有人可以告诉我如何理解这种类型的代码吗？有多个.运算符和->运算符。我忘记了班级和其他人。

这有点压倒性。

我有一个分支feature/my-changes，我已经对其进行了更改。我推送了这些更改，创建了一个拉取请求，并通过代码审查获得了关于我的更改的反馈。我的问题是：现在我看到了我需要在 github 中进行的更改（来自代码审查评论），我是否只需返回我的代码编辑器并在我的分支中进行这些更改，进行新的提交，然后再次推送？还是我应该“检查”我的分支历史的修订版并在那里进行编辑？在此代码审查之后，我不确定如何继续...