自己提交的 pull request 一直是 open，这个意思说一直在审核期吗？是不是说进入 close 了 就是审核通过了？

[no13bus]

如图：
箭头指向的是我的pull request。当时是直接被CI给merge了。

[SharkIng]

merge难道不是自动close么？

[no13bus]

@SharkIng 不是。当时我发现提交之后，几乎是同时就被merge了。但是在redis-doc里面还是没有看到的我提交的改变

[haiyang416]

Open 就是未合并或者未关闭呀。

[haiyang416]

@no13bus 如果你在 open 列表里看到了你自己的 pull request，那就是没有被合并或者关闭。

[no13bus]

@haiyang416 看来只有耐心等待了。

[haiyang416]

@no13bus 这个得看代码质量和管理者的习惯。
PR 的质量高，肯定会很快被合并的，但是如果质量不高就得看管理者的习惯了，有些管理者喜欢一直挂着，就是他觉得这个 PR 没有必要，就一直挂着，不回复，不合并，不关闭。如果你是第一次提交的话，别那么在意啦，千万别玻璃心。

[haiyang416]

另外，关闭不代表审核通过，PR 不合格就会被关闭，只有 merge 了才是通过审核了。

[SharkIng]

没有见过这种代码自动merge的，都是手动，要管理团队的人看了之后才会操作。

[no13bus]

@haiyang416
@SharkIng

我看到的是这个。是不是说只是CI build了下，但是没有merge？

[airyland]

CI 是测试若代码合并是否pass而已，不是真的合并了代码。

[haiyang416]

@no13bus CI 只是对 PR 的代码做一些检查，比如冲突、语法、测试或者代码风格等等，这个对管理者很友好，可以很明显看到 PR 是否满足一些基本条件。

[seki]

close 是你或者别人手动点的，说明这个 issue 解决了，不用再加以讨论，和代码有没有 merge 没关系

[SharkIng]

@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。然后就会到了上面说的，很多人可能看都不看，或者看了之后感觉没用也不关闭

[no13bus]

@SharkIng 多谢，确实是刚开始提交了几个pr