如何优雅地蹭贡献 :doge:

[Pastsong]

[lincanbin]

这个人给我某个 repo 发了十几个 PR ，然后一堆鸡毛蒜皮的事（每个 Commit 一般就 10 行左右），有的还有语法错误，我看得出他想混 contributor 。
于是我花了几十分钟审阅了那十几个 PR ， Merge 了几个鸡毛蒜皮的、语法错误不是特别多的。
Merge 后又花了十几分钟擦干净他代码里一一些错误（这么几行代码都有一堆错误呢）

Merge 完了，这个人跑来 PR 下面这么回复我了：

[kn007]

@lincanbin ...人不要脸天下无敌

[subpo]

哈哈哈哈哈哈哈

[Troevil]


[Troevil]

@Troevil

[pupboss]

@lincanbin 23333 他怎么在 PR 的代码里还加一行注释，说 Code by Tecflare http://www.tecflare.com

[Slienc7]

@lincanbin
Contributor:
.........XXXX with his f***ed [PR]( https://....)

[lincanbin]

@pupboss 你还跑去视奸了……
他几乎所有代码都要加这个， PR 里明确说了这个注释不让删除。
有个 PR 直接改 ReadMe ，写了自己的名字和网址和自己的简介……
我自己都没给所有代码加我自己的名字呢……

反正是鸡毛蒜皮的 PR 我也就不合并了……

[ilotuo]

哈哈哈哈哈学习了
我的 190 多个 stars 你们颤抖了吗!

[raincious]

当即决定先下手围墙

[est]

脑洞大开！

[mzer0]

你们集体在 github 上这样做的话, 老外对中国人的印象会变差的. 下次中国人发 PR, 人家就不收了......

[zsx]

@lincanbin 看到这条 Commit 感觉好无奈……

[bdbai]

完了完了， pr 刚发出去才发现重了...
http://ww3.sinaimg.com/large/a04a339djw1ezja7l0atqj20zk0k0n0g.jpg

[lincanbin]

@zsx 是我自己的疏忽， merge pull request 的时候因为那个人 commit 的代码没有合理缩进，所以我没看出那个低级的语法错误。我还以为那个人 commit 前肯定确保了自己的代码是能跑的呢……
实际上那个人的不少 PR 都是从一些地方复制过来的……


我觉得没事就别去给人提无关痛痒的 PR 了， merge 后要走的流程好多的（虽然我一般都懒得走）。

[zts1993]

简直醉了