第一次 Pull Request 被 Merge 了,感觉很开心

2017-05-17 14:19:17 +08:00
 MrFireAwayH

楼主前几天弄了个项目,因为需要所以 Fork 了一个 JS 导出 Excel 表格的插件,加了个功能后放到自己的 GitHub(无耻求 Star ) 了,测试后觉得比较完善后就给原作者提交了 Pull Request,然后刚才发现被接受了~

作者还回了老长一段,看了解释和新代码后果然感觉还是原作者比较大佬一点(汗……

大佬的回复: https://github.com/hhurz/tableExport.jquery.plugin/pull/163

因为我并不是很精通前端(只是喜欢罢了),不过因为这个事情感觉得到了大佬的肯定果然还是很开心的。

我想这大概就是开源的意义吧

3122 次点击
所在节点    程序员
11 条回复
artandlol
2017-05-17 14:27:55 +08:00
5 月 20 号了,其实原作者也只是想多钓一个萌汉子
MrFireAwayH
2017-05-17 14:28:49 +08:00
@artandlol #1 什么鬼啦!
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
这种给提交了吧 = =
MrFireAwayH
2017-05-17 14:46:19 +08:00
@littleylv #3

嘛~ 原作者自己都保留了 “ Multi Worksheets Support, Copyright (c) 2017 hejiheji001, https://github.com/hejiheji001/”

我觉得还好吧
R18
2017-05-17 14:48:58 +08:00
这就是被同行认可的快感?
MrFireAwayH
2017-05-17 15:03:41 +08:00
@R18 #5 可以这样认为~
ksaa0096329
2017-05-17 16:38:05 +08:00
@artandlol ❤️❤️❤️
MrFireAwayH
2017-05-17 16:41:47 +08:00
@ksaa0096329 #7
@artandlol #1

祝幸福
binux
2017-05-17 16:46:30 +08:00
@littleylv #3 我觉得这个改动量,获得一个署名并没有什么问题吧。
0915240
2017-05-17 17:07:18 +08:00
@R18 #8 快♂感
MrFireAwayH
2017-05-17 17:29:02 +08:00
@binux #9 谢谢跳蚤菊苣支持~

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/361948

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX