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

2015-12-04 16:46:50 +08:00
 shankee


https://github.com/apple/swift/pull/158/commits

5386 次点击
所在节点    Swift
26 条回复
loveuqian
2015-12-04 16:49:16 +08:00
毕竟提个 PR 以后面试就可以直接甩 github 了
Anteiku
2015-12-04 16:50:59 +08:00
说不定别人只是有强迫症呢?
imcoddy
2015-12-04 16:51:57 +08:00
众目睽睽啊,被 Merge 了以后就可以出去吹「哥也是给苹果打过工的人」了 233

不过这样的错误的确不应该: https://github.com/apple/swift/pull/119/files
lsmgeb89
2015-12-04 16:52:05 +08:00
@loveuqian 这有什么用,问两个问题就穿帮了。
SeanChense
2015-12-04 16:52:50 +08:00
我给 YYKit 提过两个 typo
给 @hoogle 的 V2EX 客户端 提过一个 typo

bk201
2015-12-04 17:01:27 +08:00
这个我觉得可以有啊,认真点有什么不好。
Andiry
2015-12-04 17:02:13 +08:00
毕竟没有真材实料的也就只能提这种了 PR 了
Pastsong
2015-12-04 17:02:22 +08:00
@loveuqian 提 Issue 也可以显示在 Public contribution 里
dorentus
2015-12-04 17:07:36 +08:00
但其实自己页面的 Repositories you contribute to 里面又不显示比较早时候的贡献……
Pastsong
2015-12-04 17:07:40 +08:00
@Andiry 帮别人改错字就是没有真才实学了?我英语学得比较好行不行
Pastsong
2015-12-04 17:09:31 +08:00
@dorentus 我仔细研究了一下这个应该和你提交的 commit 数量有关系
Slienc7
2015-12-04 17:19:38 +08:00
既然看到少了当然应该改
只是你想的太远了
shankee
2015-12-04 17:37:34 +08:00
@xgowex 嗯…
好吧.觉得 typo 还可以接受…这标点、好吧:我标点经常乱用”(捂脸)
yougg
2015-12-04 17:46:53 +08:00
额, 我给 google 的官方 golang 源码提过 issue 和 PR, 是不是也应该重写一笔....
WildCat
2015-12-04 17:48:17 +08:00
我的天, merge 得这么快!好评
plantain
2015-12-04 17:50:08 +08:00
看看已经 closed 的 PR ,全是 fix typo
https://github.com/apple/swift/pulls?q=is%3Apr+is%3Aclosed
phoenixlzx
2015-12-04 17:55:29 +08:00
笑点低了...
loveuqian
2015-12-04 18:03:10 +08:00
@lsmgeb89
可是我也想要两个 fix typo
sneezry
2015-12-04 18:09:47 +08:00
没什么不好的,这种还是要支持,毕竟他的 pr 让 Swift 变得更加完善了一点点,你们不能因为这个 pr 使他出现在了 Swift 贡献列表里就愤愤不平对不?
lsmgeb89
2015-12-04 18:10:09 +08:00
@loveuqian 还不如自己认真写点作品,让别人来 fix your typo ~~

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/241183

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX