问题标签 [github-codereviews]

For questions regarding programming in ECMAScript (JavaScript/JS) and its various dialects/implementations (excluding ActionScript). Note JavaScript is NOT the same as Java! Please include all relevant tags on your question; e.g., [node.js], [jquery], [json], [reactjs], [angular], [ember.js], [vue.js], [typescript], [svelte], etc.

0 投票
0 回答
2918 浏览

github - 如何在 GitHub 中请求对已提交的主分支进行代码审查?

我已经阅读了有关拉取请求的帖子并执行了代码审查,例如: “如何创建和审查 GitHub 拉取请求”

GitHub 问题:你(作为一个单独的贡献者)一直在编写你已经提交给 GitHub 的代码体有一段时间了。然后稍后在 GitHub 中,您希望其他人对提交主分支进行代码审查,并希望他们添加他们的建议。

基本上在 GitHub 上,邀请 GitHub 上的其他人对主分支中已提交的代码进行完整的代码审查,并允许添加他们的更改建议。如果请求对拉取请求进行代码审查,则具有相同的代码审查体验。

这在 GitHub 上可行吗?

谢谢

0 投票
1 回答
1539 浏览

bitbucket - 如何在 intelliJ 中查看 git 拉取请求(托管在 bitbucket 上)?

如何查看 intelliJ Idea (2018.3) 中的 git pull 请求,并在批准或拒绝之前检查编译时间、测试运行影响和其他 IDE 功能?

0 投票
1 回答
64 浏览

ios - 在 swift 4 中实施签入政策

假设我有这样的课程:

我需要实施一项政策来检查我的代码并警告我使用驼峰式大小写作为我的类名(使用 Person 而不是 person) 我如何在 Swift 中做到这一点?

0 投票
1 回答
984 浏览

git - git mv 和变化和相似性指数

当使用 重命名文件时git mv,提交将显示 rename from 和 rename to,并且在 pull request 中显示相同,这很好。但是当一个文件被git mv编辑然后更改时,看起来有一个阈值,当更改的行数超过它时,它将不再显示为重命名,而是显示为删除旧文件并添加新文件。所以我的问题是这个阈值是一个定义明确的数字吗?还有没有其他方法可以让它变得更好,主要是因为在 pull request diff 中,当两个文件不考虑重命名时,diff 不会并排显示,这使得审查变得困难。

0 投票
0 回答
270 浏览

git - git-flow 在提高 MR 进行代码审查时是不是有点模棱两可?

我是新手,git-flow有一些疑问。假设我在分支上创建了master&dev分支,git flow init并 在分支上创建feature/master-apidev分支。

一旦我完成了我所做的功能git flow feature finish master-api。功能分支被合并到我的本地dev分支中。

我的团队成员是否应该创建一个新分支dev并将 MR 提升到dev

做的时候也一样git flow release finish 1.0.0.1。代码被合并到本地master分支并返回到本地dev

devmaster(也许release/)分支应该是protected分支。

那么我们如何管理代码审查的 MR 呢?

0 投票
1 回答
172 浏览

github - 从没有访问回购权限的用户那里拉取请求审查

我经常收到邮件说类似...

您好,请审核并批准此 PR(链接)

...仅提供octobiwan的 404 服务。难免会回复...

我似乎无权访问此存储库,您能否确保已向我发送了正确的链接并且您已授予我适当的权限?

...然后听到回音...

我如何授予您权限?

想说一般...

请不要再给我发电子邮件了。您可以通过从下拉列表中选择我的姓名来请求我进行审核。如果您没有看到我的名字,那么我无法查看它,您需要先让管理员授予我访问权限

直觉上,我希望您只能请求有权完成审核的人进行审核;一些抽查似乎证实了这种行为。但是,由于我找不到任何明确证实这一点的东西,所以我想问一下互联网:

您可以向无权完成的用户请求 GitHub 代码审查吗?

...在某处是否有官方措辞说明情况是否如此?org 权限矩阵中的以下行是否尽可能明确?

0 投票
1 回答
357 浏览

git - 如何在 `git` 介质中启动对旧代码的拉取请求?

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

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

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

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

0 投票
1 回答
228 浏览

github - 如何解决 pullrequest 审查?

我创建了一个 pullrequest 并且审阅者对其进行了审核,发布了评论并请求更改,我更新了 pr 并解决了所有评论,但仍然在页面底部,它显示有人请求更改和红十字。是否有解决请求更改的选项?

0 投票
3 回答
2001 浏览

php - 不鼓励使用 echo 语言结构。(PHP)

为什么不鼓励显示“回声语言构造”的代码(代码审查工具)。

如何解决这个问题。

1在此处输入图像描述

0 投票
2 回答
453 浏览

git - 如何使用 gitlab、jenkins、git 设置代码审查工作流程?

所以我的问题是我想建立一个代码审查流程,我拥有的工具是:git,gitlab,jenkins .. 我的想法是有一种表格或清单,团队成员需要随时填写其他成员想要合并一些新代码,如果检查表填写正确并且新代码被审阅者批准,则代码应合并,否则合并请求被拒绝。

每个人都知道我该如何实现这一目标吗?或者也许有一个比清单更好的主意?

谢谢你们,

此致