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

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

  •  
  •   shankee · 2015-12-04 16:46:50 +08:00 · 4100 次点击
    这是一个创建于 1535 天前的主题,其中的信息可能已经有所发展或是发生改变。
    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   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 可以,只不过没啥意义
    关于   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   1008 人在线   最高记录 5168   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 22ms · UTC 20:40 · PVG 04:40 · LAX 12:40 · JFK 15:40
    ♥ Do have faith in what you're doing.