这是一个创建于 3571 天前的主题，其中的信息可能已经有所发展或是发生改变。
 | 1 lincanbin 2015-12-31 21:32:54 +08:00  这个人给我某个 repo 发了十几个 PR ，然后一堆鸡毛蒜皮的事（每个 Commit 一般就 10 行左右），有的还有语法错误，我看得出他想混 contributor 。 于是我花了几十分钟审阅了那十几个 PR ， Merge 了几个鸡毛蒜皮的、语法错误不是特别多的。 Merge 后又花了十几分钟擦干净他代码里一一些错误（这么几行代码都有一堆错误呢）  Merge 完了，这个人跑来 PR 下面这么回复我了：  |
 | 3 subpo PRO 哈哈哈哈哈哈哈 |
 | 4 Troevil 2015-12-31 21:43:17 +08:00 |
 | 6 pupboss 2015-12-31 21:54:13 +08:00 @lincanbin 23333 他怎么在 PR 的代码里还加一行注释，说 Code by Tecflare http://www.tecflare.com |
 | 7 Slienc7 2015-12-31 21:54:23 +08:00 |
| 8 lincanbin 2015-12-31 22:03:51 +08:00 @pupboss 你还跑去视奸了…… 他几乎所有代码都要加这个， PR 里明确说了这个注释不让删除。 有个 PR 直接改 ReadMe ，写了自己的名字和网址和自己的简介…… 我自己都没给所有代码加我自己的名字呢…… 反正是鸡毛蒜皮的 PR 我也就不合并了…… |
 | 9 ilotuo 2015-12-31 22:07:54 +08:00 哈哈哈哈哈学习了 我的 190 多个 stars 你们颤抖了吗! |
 | 10 raincious 2015-12-31 22:37:44 +08:00 当即决定先下手围墙  |
 | 11 est 2015-12-31 23:20:25 +08:00 脑洞大开！ |
 | 12 mzer0 2015-12-31 23:37:30 +08:00 你们集体在 github 上这样做的话, 老外对中国人的印象会变差的. 下次中国人发 PR, 人家就不收了...... |
 | 14 bdbai 2016-01-01 00:01:24 +08:00 完了完了， pr 刚发出去才发现重了... http://ww3.sinaimg.com/large/a04a339djw1ezja7l0atqj20zk0k0n0g.jpg |
| 15 lincanbin 2016-01-01 01:01:47 +08:00 @zsx 是我自己的疏忽， merge pull request 的时候因为那个人 commit 的代码没有合理缩进，所以我没看出那个低级的语法错误。我还以为那个人 commit 前肯定确保了自己的代码是能跑的呢…… 实际上那个人的不少 PR 都是从一些地方复制过来的…… 我觉得没事就别去给人提无关痛痒的 PR 了， merge 后要走的流程好多的（虽然我一般都懒得走）。 |
 | 16 zts1993 2016-01-01 13:27:14 +08:00 via Android 简直醉了 |
Select Language