第一次 Pull Request 被 Merge 了，感觉很开心
MrFireAwayH · hejiheji001 · 2017-05-17 14:19:17 +08:00 · 3117 次点击这是一个创建于 2551 天前的主题，其中的信息可能已经有所发展或是发生改变。
楼主前几天弄了个项目，因为需要所以 Fork 了一个 JS 导出 Excel 表格的插件，加了个功能后放到自己的 GitHub（无耻求 Star ） 了，测试后觉得比较完善后就给原作者提交了 Pull Request，然后刚才发现被接受了~
作者还回了老长一段，看了解释和新代码后果然感觉还是原作者比较大佬一点（汗……
大佬的回复： https://github.com/hhurz/tableExport.jquery.plugin/pull/163
因为我并不是很精通前端（只是喜欢罢了），不过因为这个事情感觉得到了大佬的肯定果然还是很开心的。
我想这大概就是开源的意义吧
 |
1
artandlol 2017-05-17 14:27:55 +08:00 via Android  1
5 月 20 号了，其实原作者也只是想多钓一个萌汉子
|
 |
2
MrFireAwayH OP @artandlol #1 什么鬼啦！
|
 |
3
littleylv 2017-05-17 14:38:24 +08:00
给别人提 pr 的时候不要把“-tableExport.jquery.plugin
+Forked From: https://github.com/hhurz/tableExport.jquery.plugin tableExport.jquery.plugin ” “ Copyright (c) 2015-2017 hejiheji001, https://github.com/hejiheji001/tableExport.jquery.plugin ” 这种给提交了吧 = = |
|
4
MrFireAwayH OP @littleylv #3
嘛~ 原作者自己都保留了 “ Multi Worksheets Support, Copyright (c) 2017 hejiheji001, https://github.com/hejiheji001/” 我觉得还好吧 |
 |
5
R18 2017-05-17 14:48:58 +08:00
这就是被同行认可的快感?
|
|
6
MrFireAwayH OP @R18 #5 可以这样认为~
|
 |
7
ksaa0096329 2017-05-17 16:38:05 +08:00
@artandlol ❤️❤️❤️
|
|
8
MrFireAwayH OP |
|
11
MrFireAwayH OP |
Select Language