|
loveuqianDec 4, 2015 via iPhone
1
毕竟提个 PR 以后面试就可以直接甩 github 了
|
|
AnteikuDec 4, 2015 via Android
2
说不定别人只是有强迫症呢?
|
|
imcoddyDec 4, 2015
3
众目睽睽啊,被 Merge 了以后就可以出去吹「哥也是给苹果打过工的人」了 233
不过这样的错误的确不应该: https://github.com/apple/swift/pull/119/files |
|
SeanChenseDec 4, 2015
5
|
|
6
这个我觉得可以有啊,认真点有什么不好。
|
|
AndiryDec 4, 2015 via Android
7
毕竟没有真材实料的也就只能提这种了 PR 了
|
|
dorentusDec 4, 2015
9
但其实自己页面的 Repositories you contribute to 里面又不显示比较早时候的贡献……
|
|
Slienc7Dec 4, 2015 via Android
12
既然看到少了当然应该改
只是你想的太远了 |
|
youggDec 4, 2015
14
额, 我给 google 的官方 golang 源码提过 issue 和 PR, 是不是也应该重写一笔....
|
|
WildCatDec 4, 2015
15
我的天, merge 得这么快!好评
|
|
16
看看已经 closed 的 PR ,全是 fix typo
https://github.com/apple/swift/pulls?q=is%3Apr+is%3Aclosed |
|
phoenixlzxDec 4, 2015
17
笑点低了...
|
|
sneezryDec 4, 2015 via iPhone
19
没什么不好的,这种还是要支持,毕竟他的 pr 让 Swift 变得更加完善了一点点,你们不能因为这个 pr 使他出现在了 Swift 贡献列表里就愤愤不平对不?
|
|
DelbertDec 4, 2015
21
贴图为啥要贴 mw690 ,贴 bmiddle 也好啊……
当然,最好还是贴 large 。 |
|
OuyanganDec 4, 2015
22
我给满分
|
|
ibiremeDec 4, 2015
23
我觉得。。可以做个工具来扫描 typo 。。(逃)
|
|
wsy2220Dec 4, 2015
24
Remove whitespaces...............
https://github.com/apple/swift/pull/181/files |
|
ibiremeDec 4, 2015
25
话说。。。相比 typo fix ,这个才是最丧心病狂的:
https://github.com/apple/swift/pull/17 |