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 2
这个我觉得可以有啊,认真点有什么不好。
|
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 1
看看已经 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 |