大家在 Github 上给别的 repo 提交 pull request 的策略是啥啊? - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
bwangel
V2EX    GitHub

大家在 Github 上给别的 repo 提交 pull request 的策略是啥啊?

  •  
  •   bwangel
    bwangelme 2016-04-03 06:52:46 +08:00 2540 次点击
    这是一个创建于 3551 天前的主题,其中的信息可能已经有所发展或是发生改变。

    向 V 友们请教一下,大家在 github 上向别的 repo 提交 pull request 的时如何提交的?我有两种想法:

    1. 直接提交 master

    直接在 master 分支上新建一个 commit ,然后提交 pull request 过去吗。可是如果 pull request 不被接受,那么为了以后和被 fork 的 repo 更新,岂不是 master 还要强制做一次版本回退?

    2. 新建一个 feature 分支

    新建一个 feature 分支,然后提交 pull request 过去,如果被接受了, master 会更新,然后删除了 pull request 就好了。如果没被接受,删除了这个 feature 分支就好。个人觉得这样比较好,可是感觉频繁添加删除分支会不会太麻烦了?

    大家还有其他好的策略吗?

    6 条回复    2016-04-04 09:13:23 +08:00
    goofansu
        1
    goofansu  
       2016-04-03 07:35:37 +08:00
    肯定是新建分支,删除不就点一下嘛,不想删除也不碍事
    clino
        2
    clino  
       2016-04-03 07:37:03 +08:00 via Android   1
    2
    分支都没删
    jmc891205
        3
    jmc891205  
       2016-04-03 09:42:57 +08:00
    切换分支的开销很小 就是为了让用户能频繁新建删除
    jhaohai
        4
    jhaohai  
       2016-04-03 10:04:20 +08:00 via iPhone
    我觉得 2 更好,也符合 git 的设计
    chinvo
        5
    chinvo  
       2016-04-03 10:20:41 +08:00
    2
    虽然 github 的 fork 相当于新建一个分支了,但是如果你直接在 fork 的 master 上操作,不方便跟进上游的修改。
    bwangel
        6
    bwangel  
    OP
       2016-04-04 09:13:23 +08:00
    好吧,目前看来 2 是比较好的办法了!
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     2394 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 27ms UTC 15:59 PVG 23:59 LAX 07:59 JFK 10:59
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86