V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
shankee
V2EX  ›  Swift

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

  •  
  •   shankee · 2015-12-04 16:46:50 +08:00 · 5336 次点击
    这是一个创建于 3260 天前的主题,其中的信息可能已经有所发展或是发生改变。
    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   ·   实用小工具   ·   4997 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 29ms · UTC 05:40 · PVG 13:40 · LAX 21:40 · JFK 00:40
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.