挺蛋疼的,今天在 kubernetes slack 频道里有个外国 maintainer 问了我个比较尴尬的问题:
“为什么中国人喜欢刷 PR ?” “ why chinese guys keep submitting trival pulls?”
所谓 trival pull 本指的是修改错别字,微调错误信息,无故加 unittest
当然这种 PR 本身是好的,但是中国朋友们喜欢把这些 PR 拆开了来提交,比如 typo 一刷就是小半页。外国朋友当然不理解这种现象,我小分析了下,中国人可以拿这些 PR:
就事论事,中国人当然聪明,这种投机取巧是没有成本的。虽然这点牢骚没有什么用,但是我觉得是中国技术环境的容忍导致了这样的现象,不知道是不是也该扣个打假的 tag。不知道在别的开源项目是不是也有这种现象泛滥,但是至少我在贡献的 kubernetes 有很长一段时间是这个样子的。踏踏实实,脚踏实地的参与进开源难道不好么?
分享几个链接感受下:
https://github.com/kubernetes/kubernetes/pulls?utf8=%E2%9C%93&q=is%3Apr+typo
https://groups.google.com/forum/#!topic/kubernetes-sig-docs/aapjkJ8gD1c/discussion
1
knight322 2018-08-22 18:39:06 +08:00 via Android 3
我觉得这是个人自由吧,主要是整个社会都浮躁,不注重内在,光看表面
|
2
yue9944882 OP @knight322 当然是自由,但是干扰到了开源项目正常的维护就是另一回事了
|
3
per 2018-08-22 18:41:41 +08:00 via iPhone
大部分是第一种吧,没什么能力提好的 pr,又想丰富简历。不过既不违法也不有违道德,就是有点怪怪的
|
4
yue9944882 OP @per 长此以往,怕是会加深外国人对中国开发者的刻板印象,投机的中国工程师,会这样么?
|
5
knight322 2018-08-22 18:45:43 +08:00 via Android
@yue9944882 恐怕已经如此了
|
6
ylsc633 2018-08-22 18:45:46 +08:00
惭愧啊.. 我至今还没且不会提 PR
issue 倒是会 |
7
jeneser 2018-08-22 18:46:48 +08:00 via Android 3
简历驱动 PR😉
|
8
yue9944882 OP 另外,我发现刷 PR 的项目集中出现在几家公司里。。
|
9
yue9944882 OP s/项目 /现象 /
|
10
per 2018-08-22 20:07:04 +08:00
会的吧,反正我挺反感的
@yue9944882 |
11
evlos 2018-08-22 20:13:15 +08:00
刷的人大部分都是没能力提正常 PR 的
|
12
kefengong 2018-08-22 20:13:45 +08:00 via Android
typo 还拆分就过分了
|
13
kslr 2018-08-22 20:14:02 +08:00 via Android
随着 github 的地位提高,这种情况会在世界越来越流行
|
14
guoziyan 2018-08-22 20:31:18 +08:00
刷这种 pr 有个蛋球用
|
15
yue9944882 OP > A: So if I see a PR from that same person again in my inbox, I know it's a trivial one and I won't take it "so seriously".
> B: yes, mostly chinese I think. maybe beginning of the year with a lot of typo changes. one typo at the time. some reviewers proposed to combine them into one big PR and then silence. @kefengong 上面是另外两个 maintainer 对于中国这个现象的评价。。拆分 typo 绝对是故意的 |
16
yue9944882 OP > 随着 github 的地位提高,这种情况会在世界越来越流行
世界不见得。。可见的未来中国一定会的 |
17
huclengyue 2018-08-22 21:16:37 +08:00 via Android
在 readme 里写上 XXX 不要再提交无用的 pr 了
|
18
yue9944882 OP @huclengyue 这个不会的,他们不会浪费精力较这个真,和他们沟通过甚至不希望我们中国开个 issue 讨论这件事情,要我们自行解决
|
19
ChristopherWu 2018-08-22 21:20:46 +08:00
我其实也不理解。。没什么好处
> 写进简历,哄不懂的 / 没耐心点进去看面试官 难道改 typo 的也写进去? > 赶 KPI,公司老板追求 PR 数量,逼得程序员刷 PR 哪个公司求开源项目的 pr ??晋升也不是看这个吧。。 > 做宣传甚至开始写书,对外宣称自己对开源项目贡献了几十个上百个 PR 这个也太虚了。。 |
20
yue9944882 OP @ChristopherWu
> 难道改 typo 的也写进去? 这个有的,我拿到过这样的简历上面写比如 kubernetes contributor/member 贡献了几十个 pull. 然后我在 github 上一搜下巴都要掉下来了 |
21
yue9944882 OP @ChristopherWu 而且这种现象在大厂也大有人在,有的人打着社区贡献者 /维护者的身份入职,提到社区经历就会强调自己的 PR 数量,但是打开一搜全是垃圾 PR,真的让人尴尬。。
|
22
dingyaguang117 2018-08-22 21:43:41 +08:00
我看都是译者吧,估计提交起来比较随意
|
23
JohnChiu 2018-08-22 21:47:53 +08:00
经常看到简历里面写 xxx 开源项目贡献者,都是知名项目,好奇搜一下都是修改注释、readme 中的错误单词或者不规范的代码格式的。。。这也太尬了
|
24
monsterxx03 2018-08-22 21:50:16 +08:00
早几年 OpenStack 里就有好多这种现象了
|
25
yue9944882 OP > 经常看到简历里面写 xxx 开源项目贡献者,都是知名项目,好奇搜一下都是修改注释、readme 中的错误单词或者不规范的代码格式的。。。这也太尬了
told you 🙃 |
26
sammo 2018-08-22 22:06:43 +08:00
还不是因为一些面试官吃这套?
|
27
yue9944882 OP > 还不是因为一些面试官吃这套?
悲观一点,怕面试官也是 PR 刷子 |
28
heikejia 2018-08-22 22:21:07 +08:00 1
赤裸裸的种族歧视,实际上这个行为外国佬也经常这样做,有时候改一个字母都是一个 pr,我都不好意思。。
|
29
icylogic 2018-08-22 22:37:55 +08:00 via iPhone
我只给一个开源项目提交过比较多的 pr,不过不敢写进简历里(也不好意思)
|
30
yue9944882 OP > 赤裸裸的种族歧视
不是的,他们还特意澄清了下这点,事实摆在那里 >> About how others feel about it: I don't think they think it in terms of "all Chinese people are sending trivial PRs". I think it becomes more of "@-username is sending trivial PRs". |
31
yue9944882 OP [https://github.com/kubernetes/kubernetes/pull/59464]( https://github.com/kubernetes/kubernetes/pull/59464)
看这个 PR 下面的 reference,都是外国维护者一个一个手动关掉的,这样的 trival pull 像苍蝇虱子一样。说无关痛痒,不好意思,人家真没有那么大度 |
32
huiyifyj 2018-08-22 23:06:22 +08:00 via Android
没有提交 10 %怎么好意思写进简历?
|
33
nooper 2018-08-22 23:27:03 +08:00 via iPad
我觉得不如自己造一个牛比的
|
34
hanru 2018-08-22 23:30:49 +08:00 via Android
这和某些大学教授向 Nature 投稿读者来信,然后宣称在国际顶级期刊发表了学术论文有异曲同工之妙。
|
35
ysc3839 2018-08-22 23:33:23 +08:00 via Android
我想知道这跟拼夕夕有啥关系?
|
36
yue9944882 OP |
37
deadEgg 2018-08-22 23:59:38 +08:00
浮躁
人人都想一蹴而就,如果不付出努力就能获得,谁都想这么做。 |
38
camork 2018-08-23 00:27:58 +08:00 via iPhone
有点好奇为什么这个库这么多 typo😂
|
39
studyro 2018-08-23 01:46:37 +08:00 via iPhone
类似情况是很多应聘者的博客。
刚好在每次换工作前频繁更新数篇很粗糙的文章,然后一年两年内就不更新了直到下次换工作前。 也是挺有意思。 |
40
wspsxing 2018-08-23 02:16:14 +08:00 via Android
我曾经看到过一个外国人的 pr,™的就给我加了个换行符。。fix typo 都算不上,没道理接受。
讲真正经的 fix typo 对项目来说是有用的, 但是一个 typo 有多少处就分为多少个就过分了 |
41
liyuanba 2018-08-23 06:54:00 +08:00
|
42
mathzhaoliang 2018-08-23 08:09:44 +08:00
我的项目
https://github.com/neozhaoliang/pywonderland 两年来就没人提过正经的 pull requests, 全是改 typo, add travis, add docker 这种。 |
43
abmin521 2018-08-23 08:32:15 +08:00 via Android
利益相关 被合并之后会有利益
楼主说的全对 简历驱动 公司宣传(客户又不懂) 我还见有公司的招标书上写本公司全部 211 毕业,名校情结哪里都存在的 原谅我斗胆猜一下,国内能给 k8s 核心代码做贡献没 100 个,有的人即使有实力也有别的事情要忙, 阿里在忙着 pouch 腾讯和百度只是研究而已 不说了 还是好好看源码去了 |
44
bk201 2018-08-23 08:43:51 +08:00
这个刷了如果遇到仔细的面试官不是要扣分嘛
|
45
myl0204 2018-08-23 08:47:17 +08:00
水平比较差,只能改改 typo 这样子,但是我是不会写进简历的
|
46
marvinGet 2018-08-23 08:50:06 +08:00
题主如果有什么想不通的,记住我的真言:这是在中国!
|
47
liuzhedash 2018-08-23 09:08:43 +08:00
@sammo #26
上有所好,下必甚焉。 新晋菜鸡程序员代码都看得不太明白,能提得了什么靠谱的 PR ? 所以都是自作自受:那些能提靠谱 PR 的,并且以此打击新晋菜鸡的人,就只能看着自己参与的项目被辣鸡 PR 淹没。 |
48
portgas99 2018-08-23 09:09:19 +08:00 via iPhone
拼夕夕呢?
|
49
xomix 2018-08-23 09:11:31 +08:00
如果我是作者,起码这是有所贡献的人了,总比一些整天 fork 或者下载了屁都不放一个就走的人强点儿。
个人作者当然什么样的人都有有人反感这些,还有些个人作者反感有 PR 呢。 如果是专业一些准备产品销售的公司,这样的人他巴不得多来一些,自己的程序员可以放松一些专心开发。 |
50
98jiang 2018-08-23 09:35:16 +08:00
想问下 typo 是指修改格式吗。。
|
51
wu67 2018-08-23 09:39:55 +08:00
就项目所有者来说, 还是欢迎这些的, 毕竟人的精力有限, 细节没发面面俱到. 但是就个人来说, 我是非常反感这些刷 issue pr 和小绿点的, 特别是绿点, 看到就烦
|
52
weizhen199 2018-08-23 09:40:20 +08:00
@yue9944882 恐怕就是这样没错
|
53
rocksolid 2018-08-23 09:40:47 +08:00
关键是人多,你觉得老是看到中国人
|
54
zcore 2018-08-23 09:49:40 +08:00
社会竞争激烈抬高了这种行为的收益,虽然只是一时虚高,但相比其低成本,可以说是一本万利了。同时它也降低了正当竞争途径的收益,逼得很多人不得不做同样的事,这样反过来促进这种不正当竞争,形成一种正循环。说白了,激烈的社会竞争才是本因,在生存压力面前,任何道德都可以抛之脑后,乃人之本性。
|
55
nervdy 2018-08-23 09:50:37 +08:00 2
|
56
itqls 2018-08-23 09:53:27 +08:00
我都不好意思把我提过 issue 和参与讨论过的项目挂到首页的,自己有没有贡献心里没逼数么
|
57
cncqw 2018-08-23 10:00:10 +08:00 1
@xomix #49
要放什么屁? 没用怎么知道好不好, 你的意思是 github 搞个评论区,要评论才能下载,评论里刷屏回复 up 主好人 项目好用给 star 就是对项目的认可和对作者表示致敬,有 bug 自然会提 issue 或者 pr “如果我是作者,起码这是有所贡献的人” 一个项目别人还没用过你就开始在意别人有没有贡献,那你就不要开源,没人强迫你 对了,有些人项目里还喜欢在 readme 里放个收款二维码,可以说是土味十足 编程本是种乐趣,我一直认为功利心太重的人往往是写不出好代码的 |
59
abmin521 2018-08-23 10:01:51 +08:00
@ChristopherWu
> 写进简历,哄不懂的 / 没耐心点进去看面试官 难道改 typo 的也写进去? >>> 他们只会写曾给 Google 的 kubernetes 提交 X 个 PR,并成功被 merged > 赶 KPI,公司老板追求 PR 数量,逼得程序员刷 PR 哪个公司求开源项目的 pr ??晋升也不是看这个吧。。 >>> 某些技术公司当然也宣传一下自己的开源精神,参考贵司的 alisql dubbo 等等 > 做宣传甚至开始写书,对外宣称自己对开源项目贡献了几十个上百个 PR 这个也太虚了。。 >>> 这个上面解释过,客户一般不太懂技术,不会去看你提的什么 pr |
60
hhsuan 2018-08-23 10:07:27 +08:00
合理利用规则,很正常啊。不知道这些老外叽叽歪歪什么,有本事你们要求 github 修改规则啊 。
|
61
xianxiaobo 2018-08-23 10:19:12 +08:00
看到了最后才想起我是因为拼夕夕点进来的
|
62
zxxufo008 2018-08-23 10:44:58 +08:00
所以楼主用拼夕夕做标题算不算自己也是一种拼夕夕行为?
|
63
passerbytiny 2018-08-23 10:58:55 +08:00
恶意刷的不讨论。我只说说无意刷的原因:绝大部分公司,在软件开发没有想到或者刻意忽略“过程”,或者用一个词代替“随想随改”。
多数人(尤其是非编码人员)的思维里是这样的:有问题,赶紧改,改完赶紧提交,我要看结果;改错了,赶紧再改回来;我下午 5 点有个想法,你下班前就给做出来;我不管理过程啥样,我只看结果。这种思维,加上一般很少做提交评审,是必然会导致冗余甚至混乱的提交历史的。 |
64
omzsl 2018-08-23 11:07:15 +08:00
您还真会蹭热点
|
65
yue9944882 OP > 社会竞争激烈抬高了这种行为的收益,虽然只是一时虚高,但相比其低成本,可以说是一本万利了。同时它也降低了正当竞争途径的收益,逼得很多人不得不做同样的事,这样反过来促进这种不正当竞争,形成一种正循环。说白了,激烈的社会竞争才是本因,在生存压力面前,任何道德都可以抛之脑后,乃人之本性。
👍🏻👍🏻有见解 |
66
PureWhiteWu 2018-08-23 11:20:52 +08:00 1
你们有点没考虑到:大多数人都忙于工作,中国程序员加班严重,连自己的工作都做不完的。
大家都想给开源项目做点贡献,想当个 contributor,无奈没有时间精力做大的修改,想要在 k8s 社区推动一件事情的周期太长和需要花费的时间精力又太多,而且你不脸熟的话稍微大一点的事情不会让你做。 所以只能是看到一个 typo 之类的,就提交一下,混个 contributor,也是对开源项目尽了一点力。 中国程序员又不像外国程序员,每天 5 点准时下班,有的是个人的时间能够花在开源项目上。 当然,一定有你说的那种刷 pr 刷简历的人,但我相信大部分人还是好的,只是确实没有时间精力去推动一些更大的 pr。 |
67
sethverlo 2018-08-23 11:24:53 +08:00 1
中枪…之前用 Google API, 看到少了个空格,就随手加了,我就是看着不舒服啊 😂
|
68
openbsd 2018-08-23 11:25:43 +08:00
拼歹歹也就是卖点寨货,然后上市圈钱了,拉了一堆的仇恨
|
69
yue9944882 OP @sethverlo 我还是得澄清下.. 反对的是拆开 trival PR 提交的现象,提交 trival pull 本身很正常的我也会做
|
70
Felldeadbird 2018-08-23 11:53:16 +08:00
目前是 踏实做事 的人 得不到赏识。 走偏门捷径的人反而上位。 貌似古今中外都是如此。
|
71
Akagi201 2018-08-23 12:48:41 +08:00
踏实做事的人得不到赏识, 走偏门捷径的人反而上位 +1
|
72
hunger1516 2018-08-23 12:49:55 +08:00 1
看到路边一个烟头,你顺手扔进垃圾桶,然后就有一些人背后戳脊梁骨,这人真爱作秀,有本事把路边的烟头全捡了
要骂倒是可以骂那些捡一个烟头还拍照发朋友圈的人,以及那些在后面乱戳脊梁骨的戏精 |
73
aljun 2018-08-23 13:15:10 +08:00
我自己的项目也会莫名其妙收到 typo 的 pr。。而且语气还很不客气(而且我项目都英文的,为什么 PR 要中文。。
我给其他项目提的 PR 都是他们的写在 readme 的 feature 啊(当然 readme 一起 pr 了 orz |
75
yue9944882 OP > 要骂倒是可以骂那些捡一个烟头还拍照发朋友圈的人,以及那些在后面乱戳脊梁骨的戏精
吐槽的是把烟头捡走熬汤拿出来卖的行为 |
76
zzzzzzZ 2018-08-23 14:24:10 +08:00 1
简历的目的倒是有可能,培训装逼的动机太少了
曾经圈内人士可以告诉你,一个学了 2 个月 html 的厨子,去腾讯大厦门口穿一身西装扶着 logo 拍张照,转身就是鹅厂资深员工。 拿两个静态 html 页,一个普通的人物照,另一个比前面那张多了几个红圈,一个 button 直接跳转,这就是人脸识别。 画几个柱状图,横纵轴放一些很装逼很厉害的名称,js 做一点动态效果(估计是厨子找同事帮忙做的),这就是大数据分析。 一个这样的厨子在痰州一晚上十几万至几十万的销售额,月薪 30k+提成,带不了学生只装逼+招生,教学交给其他人 github 这种装逼太 LOW 了,而且和受骗对象的知识程度不相符 |
77
kangkang 2018-08-23 14:27:37 +08:00 1
"合理利用规则,很正常啊。"----这种言论都在 v2 出现了,这里的用户群越来越迷了
|
78
est 2018-08-23 15:04:14 +08:00
该 1 个字的 PR 我也提过。
我脸厚。 |
79
Geo200 2018-08-23 15:05:07 +08:00
这种现象三年前就有了,另我印象最深刻的一个人是叫秋风,当年看到她的项目都是 jQuery、angular 之类的,github 上好看得很,实际上也是改下 md 提交 pr,不久之后被知乎上的人揭穿,后来换小号继续混了
|
80
miniwade514 2018-08-23 15:06:50 +08:00
把 1~2 分的功力,包装出 3~4 分的效果,进入一家 5~6 分水平的公司。边学边干,能力总会提升。过段时间再跳个槽,薪水总会上涨。循环往复,螺旋上升。
当然楼主帖子里说的人不会都是这种情况,但我猜有不少人打着这种算盘。 |
81
ugu 2018-08-23 15:29:51 +08:00
这样就可以说我也参加过开源项目了
|
82
silenceshell 2018-08-23 15:37:14 +08:00
还好我除了 fix typo 以外还有一些正经功能的 PR,要不然还不敢说下面这句:
一些 typo 看到了顺手改了,我觉得也无可厚非吧。 |
83
0x8192dd 2018-08-23 15:57:19 +08:00
PR 多多 PR 多多,PR 多工作多,每个大项目都要 PR 多多,一个错字也要 PR 多多,PR 多多😂
|
84
sakeven 2018-08-23 16:06:14 +08:00 2
|
85
yue9944882 OP > @sakeven 针对人没意义的,就事论事
|
88
deepkolos 2018-08-23 18:22:02 +08:00
唉, 之前也是刷 github commit, 开始还好, 还是有输出, 但是后面, 开始有点为了刷量而 commit 了, 所以自己还是停止了这样自欺欺人的行为了
|
89
abmin521 2018-08-23 19:01:50 +08:00 1
|
90
persimmon 2018-08-23 19:04:09 +08:00
from The Rails Doctrine
blabla blabla ...... You never know when the next person who starts just fixing a misspelling in the documentation ends up implementing the next great feature. But you stand a chance to find out if you smile and say thank you for whatever small contribution that gets the motivation flowing. 有人给你贡献 PR 是好事情,但划水 PR 也是很容易看出来的 |
91
persimmon 2018-08-23 19:21:07 +08:00
实际原则上,无论事情大小,中国人对于犯错容忍度是很高的,更别说是做事方式这种鸡毛蒜皮,毕竟面试官一看他也的确是在贡献 PR
|
92
mengzhuo 2018-08-24 07:35:37 +08:00 via iPhone
typo 提交老外也不少啊,说明项目 review 没做好。
Go 每个 CL 都消灭了不少 typo。 |
93
qiuqiuer 2018-08-24 11:09:16 +08:00 via Android
圈钱召唤师 培训基地?
|
94
yue9944882 OP > typo 提交老外也不少啊,说明项目 review 没做好。
中国是把 trival pull 拆了交,而且很多很多人这样做.. 看我上面贴的 PR,一个自动修正 typo 的脚本 conflict 了多少交 typo 的中国人?下面有外国人么? |
95
mary9 2018-08-24 23:49:31 +08:00
之前不是有个大厂的人在某个项目上贡献了一个空格?我也是震惊了,关键是本人还觉着很自豪,说一个空格也是贡献😲 另外我觉着,这种人也不分国界的,flv.js 的作者 qq 说,有个老外想为 flv 贡献一个项目吉祥物----一个 icon,哈哈,亏想的出来😁
|
96
mars0prince 2018-09-03 14:06:15 +08:00
公司风格吧,很多公司的要求就是加个空格都要单独提个 commit,这种 pr 其实是文化差异?
|