自己提交的 pull request 一直是 open，这个意思说一直在审核期吗？是不是说进入 close 了 就是审核通过了？
no13bus · 2015-04-20 11:01:57 +08:00 · 4425 次点击这是一个创建于 3482 天前的主题，其中的信息可能已经有所发展或是发生改变。
如图：
箭头指向的是我的pull request。当时是直接被CI给merge了。
箭头指向的是我的pull request。当时是直接被CI给merge了。
 |
1
SharkIng 2015-04-20 11:13:59 +08:00
merge难道不是自动close么？
|
 |
2
no13bus OP @SharkIng 不是。当时我发现提交之后，几乎是同时就被merge了。但是在redis-doc里面还是没有看到的我提交的改变
|
 |
3
haiyang416 2015-04-20 11:30:27 +08:00
Open 就是未合并或者未关闭呀。
|
|
4
haiyang416 2015-04-20 11:31:16 +08:00  1
@no13bus 如果你在 open 列表里看到了你自己的 pull request，那就是没有被合并或者关闭。
|
|
5
no13bus OP @haiyang416 看来只有耐心等待了。
|
|
6
haiyang416 2015-04-20 11:35:47 +08:00
@no13bus 这个得看代码质量和管理者的习惯。
PR 的质量高，肯定会很快被合并的，但是如果质量不高就得看管理者的习惯了，有些管理者喜欢一直挂着，就是他觉得这个 PR 没有必要，就一直挂着，不回复，不合并，不关闭。如果你是第一次提交的话，别那么在意啦，千万别玻璃心。 |
|
7
haiyang416 2015-04-20 11:37:28 +08:00
另外，关闭不代表审核通过，PR 不合格就会被关闭，只有 merge 了才是通过审核了。
|
|
8
SharkIng 2015-04-20 11:47:38 +08:00
没有见过这种代码自动merge的，都是手动，要管理团队的人看了之后才会操作。
|
|
9
no13bus OP |
 |
10
airyland 2015-04-20 12:30:18 +08:00 1
CI 是测试若代码合并是否pass而已，不是真的合并了代码。
|
|
11
haiyang416 2015-04-20 12:32:45 +08:00
@no13bus CI 只是对 PR 的代码做一些检查，比如冲突、语法、测试或者代码风格等等，这个对管理者很友好，可以很明显看到 PR 是否满足一些基本条件。
|
 |
12
seki 2015-04-20 12:33:22 +08:00
close 是你或者别人手动点的，说明这个 issue 解决了，不用再加以讨论，和代码有没有 merge 没关系
|
|
13
SharkIng 2015-04-20 12:53:49 +08:00
@no13bus 这么说来你还是对于Github的处理方式不清楚。在提交Pull request的时候，Github会根据commit的代码不同进行一次审查，如果没有conflict， 那么表明可以merge，这里的这句话意思就是提示作者可以merge，没有冲突
所谓authomatically merged，不是说自动merge（请注意是Can be， 而不是is或者have been）而是说没有conflict的情况下直接在Github点击一个键就可以自动merge。在有conflict的情况下，需要手动Merge，所谓手动不是说作者手动按键，而是需要在自己机子上使用Git来处理Conflict然后再Merge到一起。 所以，至少我知道的是这样的，Github上你提交了Pull request如果没有conflict都会有这么样一句话，如果有的话提示就是请到Git中手动Merge （我一会儿找个图看看）在提示这句话了之后，那么作者（Repo所有者）需要通过手动按键来选择是Merge还是不Merge。然后就会到了上面说的，很多人可能看都不看，或者看了之后感觉没用也不关闭 |
Select Language