V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX 提问指南
no13bus
V2EX  ›  问与答

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

  •  
  •   no13bus · 2015-04-20 11:01:57 +08:00 · 4268 次点击
    这是一个创建于 3265 天前的主题,其中的信息可能已经有所发展或是发生改变。
    如图:
    箭头指向的是我的pull request。当时是直接被CI给merge了。

    14 条回复    2015-04-20 13:15:42 +08:00
    SharkIng
        1
    SharkIng  
       2015-04-20 11:13:59 +08:00
    merge难道不是自动close么?
    no13bus
        2
    no13bus  
    OP
       2015-04-20 11:23:18 +08:00
    @SharkIng 不是。当时我发现提交之后,几乎是同时就被merge了。但是在redis-doc里面还是没有看到的我提交的改变
    haiyang416
        3
    haiyang416  
       2015-04-20 11:30:27 +08:00
    Open 就是未合并或者未关闭呀。
    haiyang416
        4
    haiyang416  
       2015-04-20 11:31:16 +08:00   ❤️ 1
    @no13bus 如果你在 open 列表里看到了你自己的 pull request,那就是没有被合并或者关闭。
    no13bus
        5
    no13bus  
    OP
       2015-04-20 11:32:06 +08:00
    @haiyang416 看来只有耐心等待了。
    haiyang416
        6
    haiyang416  
       2015-04-20 11:35:47 +08:00
    @no13bus 这个得看代码质量和管理者的习惯。
    PR 的质量高,肯定会很快被合并的,但是如果质量不高就得看管理者的习惯了,有些管理者喜欢一直挂着,就是他觉得这个 PR 没有必要,就一直挂着,不回复,不合并,不关闭。如果你是第一次提交的话,别那么在意啦,千万别玻璃心。
    haiyang416
        7
    haiyang416  
       2015-04-20 11:37:28 +08:00
    另外,关闭不代表审核通过,PR 不合格就会被关闭,只有 merge 了才是通过审核了。
    SharkIng
        8
    SharkIng  
       2015-04-20 11:47:38 +08:00
    没有见过这种代码自动merge的,都是手动,要管理团队的人看了之后才会操作。
    no13bus
        9
    no13bus  
    OP
       2015-04-20 12:27:33 +08:00
    @haiyang416
    @SharkIng

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

    airyland
        10
    airyland  
       2015-04-20 12:30:18 +08:00   ❤️ 1
    CI 是测试若代码合并是否pass而已,不是真的合并了代码。
    haiyang416
        11
    haiyang416  
       2015-04-20 12:32:45 +08:00
    @no13bus CI 只是对 PR 的代码做一些检查,比如冲突、语法、测试或者代码风格等等,这个对管理者很友好,可以很明显看到 PR 是否满足一些基本条件。
    seki
        12
    seki  
       2015-04-20 12:33:22 +08:00
    close 是你或者别人手动点的,说明这个 issue 解决了,不用再加以讨论,和代码有没有 merge 没关系
    SharkIng
        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。然后就会到了上面说的,很多人可能看都不看,或者看了之后感觉没用也不关闭
    no13bus
        14
    no13bus  
    OP
       2015-04-20 13:15:42 +08:00
    @SharkIng 多谢,确实是刚开始提交了几个pr
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   952 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 29ms · UTC 21:12 · PVG 05:12 · LAX 14:12 · JFK 17:12
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.