抢着给 Swift 提 PR 的, 丧心病狂到标点都不放过的节奏吗? - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
shankee
V2EX    wift

抢着给 Swift 提 PR 的, 丧心病狂到标点都不放过的节奏吗?

  •  
  •   shankee 2015-12-04 16:46:50 +08:00 5692 次点击
    这是一个创建于 3600 天前的主题,其中的信息可能已经有所发展或是发生改变。
    26 条回复    2015-12-04 23:25:50 +08:00
    loveuqian
        1
    loveuqian  
       2015-12-04 16:49:16 +08:00 via iPhone
    毕竟提个 PR 以后面试就可以直接甩 github 了
    Anteiku
        2
    Anteiku  
       2015-12-04 16:50:59 +08:00 via Android
    说不定别人只是有强迫症呢?
    imcoddy
        3
    imcoddy  
       2015-12-04 16:51:57 +08:00
    众目睽睽啊,被 Merge 了以后就可以出去吹「哥也是给苹果打过工的人」了 233

    不过这样的错误的确不应该: https://github.com/apple/swift/pull/119/files
    lsmgeb89
        4
    lsmgeb89  
       2015-12-04 16:52:05 +08:00
    @loveuqian 这有什么用,问两个问题就穿帮了。
    SeanChense
        5
    SeanChense  
       2015-12-04 16:52:50 +08:00
    我给 YYKit 提过两个 typo
    给 @hoogle 的 V2EX 客户端 提过一个 typo

    bk201
        6
    bk201  
       2015-12-04 17:01:27 +08:00   2
    这个我觉得可以有啊,认真点有什么不好。
    Andiry
        7
    Andiry  
       2015-12-04 17:02:13 +08:00 via Android
    毕竟没有真材实料的也就只能提这种了 PR 了
    Pastsong
        8
    Pastsong  
       2015-12-04 17:02:22 +08:00
    @loveuqian 提 Issue 也可以显示在 Public contribution 里
    dorentus
        9
    dorentus  
       2015-12-04 17:07:36 +08:00
    但其实自己页面的 Repositories you contribute to 里面又不显示比较早时候的贡献……
    Pastsong
        10
    Pastsong  
       2015-12-04 17:07:40 +08:00   1
    @Andiry 帮别人改错字就是没有真才实学了?我英语学得比较好行不行
    Pastsong
        11
    Pastsong  
       2015-12-04 17:09:31 +08:00
    @dorentus 我仔细研究了一下这个应该和你提交的 commit 数量有关系
    Slienc7
        12
    Slienc7  
       2015-12-04 17:19:38 +08:00 via Android
    既然看到少了当然应该改
    只是你想的太远了
    shankee
        13
    shankee  
    OP
       2015-12-04 17:37:34 +08:00 via Android
    @xgowex 嗯…
    好吧.觉得 typo 还可以接受…这标点、好吧:我标点经常乱用”(捂脸)
    yougg
        14
    yougg  
       2015-12-04 17:46:53 +08:00
    额, 我给 google 的官方 golang 源码提过 issue 和 PR, 是不是也应该重写一笔....
    WildCat
        15
    WildCat  
       2015-12-04 17:48:17 +08:00
    我的天, merge 得这么快!好评
    plantain
        16
    plantain  
       2015-12-04 17:50:08 +08:00   1
    看看已经 closed 的 PR ,全是 fix typo
    https://github.com/apple/swift/pulls?q=is%3Apr+is%3Aclosed
    phoenixlzx
        17
    phoenixlzx  
       2015-12-04 17:55:29 +08:00
    笑点低了...
    loveuqian
        18
    loveuqian  
       2015-12-04 18:03:10 +08:00
    @lsmgeb89
    可是我也想要两个 fix typo
    sneezry
        19
    sneezry  
       2015-12-04 18:09:47 +08:00 via iPhone
    没什么不好的,这种还是要支持,毕竟他的 pr 让 Swift 变得更加完善了一点点,你们不能因为这个 pr 使他出现在了 Swift 贡献列表里就愤愤不平对不?
    lsmgeb89
        20
    lsmgeb89  
       2015-12-04 18:10:09 +08:00
    @loveuqian 还不如自己认真写点作品,让别人来 fix your typo ~~
    Delbert
        21
    Delbert  
       2015-12-04 19:36:29 +08:00
    贴图为啥要贴 mw690 ,贴 bmiddle 也好啊……
    当然,最好还是贴 large 。
    Ouyangan
        22
    Ouyangan  
       2015-12-04 21:57:55 +08:00
    我给满分
    ibireme
        23
    ibireme  
       2015-12-04 23:10:56 +08:00
    我觉得。。可以做个工具来扫描 typo 。。(逃)
    wsy2220
        24
    wsy2220  
       2015-12-04 23:11:16 +08:00
    Remove whitespaces...............
    https://github.com/apple/swift/pull/181/files
    ibireme
        25
    ibireme  
       2015-12-04 23:23:10 +08:00
    话说。。。相比 typo fix ,这个才是最丧心病狂的:
    https://github.com/apple/swift/pull/17
    Andiry
        26
    Andiry  
       2015-12-04 23:25:50 +08:00 via Android
    @Pastsong 可以,只不过没啥意义
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     4480 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 27ms UTC 04:06 PVG 12:06 LAX 21:06 JFK 00:06
    Do have faith in what you're doing.
    ubao 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