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

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

› Swift Blog

› Swift.org

› Swift Programming on Medium

› Vapor

› Perfect Web Framework

› Kitura Web Framework

这是一个创建于 3310 天前的主题，其中的信息可能已经有所发展或是发生改变。

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

标点

丧心病狂

Swift

放过

26 条回复 • 2015-12-04 23:25:50 +08:00

loveuqian

2015-12-04 16:49:16 +08:00 via iPhone

毕竟提个 PR 以后面试就可以直接甩 github 了

Anteiku

2015-12-04 16:50:59 +08:00 via Android

说不定别人只是有强迫症呢？

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 via Android

毕竟没有真材实料的也就只能提这种了 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 via Android

既然看到少了当然应该改
只是你想的太远了

shankee

2015-12-04 17:37:34 +08:00 via Android

@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 via iPhone

没什么不好的，这种还是要支持，毕竟他的 pr 让 Swift 变得更加完善了一点点，你们不能因为这个 pr 使他出现在了 Swift 贡献列表里就愤愤不平对不？

lsmgeb89

2015-12-04 18:10:09 +08:00

@loveuqian 还不如自己认真写点作品，让别人来 fix your typo ~~

Delbert

2015-12-04 19:36:29 +08:00

贴图为啥要贴 mw690 ，贴 bmiddle 也好啊……
当然，最好还是贴 large 。

Ouyangan

2015-12-04 21:57:55 +08:00

我给满分

ibireme

2015-12-04 23:10:56 +08:00

我觉得。。可以做个工具来扫描 typo 。。（逃）

wsy2220

2015-12-04 23:11:16 +08:00

Remove whitespaces...............
https://github.com/apple/swift/pull/181/files

ibireme

2015-12-04 23:23:10 +08:00

话说。。。相比 typo fix ，这个才是最丧心病狂的：
https://github.com/apple/swift/pull/17

Andiry

2015-12-04 23:25:50 +08:00 via Android

@Pastsong 可以，只不过没啥意义