1
yyfearth 2015-03-08 05:53:26 +08:00
我觉得 PR就是为了方便代码审查 相当于一个特殊的branch
另外 “原repo上开新的branch” 也可以创建PR 方便代码审查 如果你们是同一个组织的 同一个repo方便管理和维护 但是如果是开源社区 你不方便或者不允许直接参与repo的管理 或者需要维护自己的branch 那么久fork吧 |
2
yyfearth 2015-03-08 05:56:02 +08:00
我记得一开始PR的出现 就是为了fork之后任然可以像用branch一样合并
方便不同fork repos直接的代码交流 如果不正确的话 请指明 |
4
liubiantao 2015-03-08 09:27:29 +08:00
既然你提到了小绿点,那应该是在 github 上且开源的项目。
第一种是针对未授权或者不信任的陌生人。 如果是同项目组的或者你确信他不会给你搞破坏的人,一般都不用第一种。 第二种是显示小绿点的,我刚才测试过了,而且 github 帮助也明确说了这种情况: >> You are a collaborator on the repository or are a member of the organization that owns the repository. https://help.github.com/articles/why-are-my-contributions-not-showing-up-on-my-profile/ 如果没显示小绿点,你看看是不是不符合其他的条件。 其实还有一种方法,你们建立一个 organization ,项目组都加入这个 organization 。 |
5
liubiantao 2015-03-08 09:29:44 +08:00 1
哦,对了,只有更改默认分支(一般是 master) 还有 gh-pages 才会显示小绿点,所以等你的 branch merge 到 master 之后,小绿点就出来了。
|
6
msg7086 2015-03-08 11:11:58 +08:00
我们以前单位里开发开源软件,是每个人一个branch,完成以后开PR,测试通过&审查通过以后统一merge。
和小绿点没关系吧?只要做好账号关联应该都有点? |
7
wittyfox 2015-03-08 12:13:41 +08:00
|
8
spacewander 2015-03-08 13:52:12 +08:00
第一种适合添加小特性或者bug fix,第二种适合去尝试一些新功能。
|
9
chunyang 2015-03-08 15:09:52 +08:00
你有权限的话,也可以开个 feature 分支,然后提交 PR,过一段时间自己或者让别的有权限的人 merge 掉你的 PR,幸运的话,有人会帮你做 review。
这样的话 commit 历史中会多一个 merge 记录(merge v.s. rebase)。 P.S. 开个 PR 本身 Github 似乎也算作一个绿点,同样的,开个 issue 也算 |