shankee
V2EX  ›  Swift

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

  •  
  •   shankee · Dec 4, 2015 · 6511 views
    This topic created in 3992 days ago, the information mentioned may be changed or developed.
    26 replies  •  2015-12-04 23:25:50 +08:00
    loveuqian
    loveuqian
    Dec 4, 2015 via iPhone
    1
    毕竟提个 PR 以后面试就可以直接甩 github 了
    Anteiku
    Anteiku
    Dec 4, 2015 via Android
    2
    说不定别人只是有强迫症呢?
    imcoddy
    imcoddy
    Dec 4, 2015
    3
    众目睽睽啊,被 Merge 了以后就可以出去吹「哥也是给苹果打过工的人」了 233

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

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