![]() | 1 loveuqian 2015-12-04 16:49:16 +08:00 via iPhone 毕竟提个 PR 以后面试就可以直接甩 github 了 |
![]() | 2 Anteiku 2015-12-04 16:50:59 +08:00 via Android 说不定别人只是有强迫症呢? |
3 imcoddy 2015-12-04 16:51:57 +08:00 众目睽睽啊,被 Merge 了以后就可以出去吹「哥也是给苹果打过工的人」了 233 不过这样的错误的确不应该: https://github.com/apple/swift/pull/119/files |
![]() | 5 SeanChense 2015-12-04 16:52:50 +08:00 |
![]() | 6 bk201 2015-12-04 17:01:27 +08:00 ![]() 这个我觉得可以有啊,认真点有什么不好。 |
![]() | 7 Andiry 2015-12-04 17:02:13 +08:00 via Android 毕竟没有真材实料的也就只能提这种了 PR 了 |
![]() | 9 dorentus 2015-12-04 17:07:36 +08:00 但其实自己页面的 Repositories you contribute to 里面又不显示比较早时候的贡献…… |
![]() | 12 Slienc7 2015-12-04 17:19:38 +08:00 via Android 既然看到少了当然应该改 只是你想的太远了 |
![]() | 14 yougg 2015-12-04 17:46:53 +08:00 额, 我给 google 的官方 golang 源码提过 issue 和 PR, 是不是也应该重写一笔.... |
![]() | 15 WildCat 2015-12-04 17:48:17 +08:00 我的天, merge 得这么快!好评 |
![]() | 16 plantain 2015-12-04 17:50:08 +08:00 ![]() 看看已经 closed 的 PR ,全是 fix typo https://github.com/apple/swift/pulls?q=is%3Apr+is%3Aclosed |
![]() | 17 phoenixlzx 2015-12-04 17:55:29 +08:00 笑点低了... |
![]() | 19 sneezry 2015-12-04 18:09:47 +08:00 via iPhone 没什么不好的,这种还是要支持,毕竟他的 pr 让 Swift 变得更加完善了一点点,你们不能因为这个 pr 使他出现在了 Swift 贡献列表里就愤愤不平对不? |
![]() | 21 Delbert 2015-12-04 19:36:29 +08:00 贴图为啥要贴 mw690 ,贴 bmiddle 也好啊…… 当然,最好还是贴 large 。 |
![]() | 22 Ouyangan 2015-12-04 21:57:55 +08:00 我给满分 |
23 ibireme 2015-12-04 23:10:56 +08:00 我觉得。。可以做个工具来扫描 typo 。。(逃) |
24 wsy2220 2015-12-04 23:11:16 +08:00 Remove whitespaces............... https://github.com/apple/swift/pull/181/files |
25 ibireme 2015-12-04 23:23:10 +08:00 话说。。。相比 typo fix ,这个才是最丧心病狂的: https://github.com/apple/swift/pull/17 |