V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
• 请不要在回答技术问题时复制粘贴 AI 生成的内容
MrFireAwayH
V2EX  ›  程序员

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

  •  2
     
  •   MrFireAwayH ·
    hejiheji001 · 2017-05-17 14:19:17 +08:00 · 3194 次点击
    这是一个创建于 2772 天前的主题,其中的信息可能已经有所发展或是发生改变。

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

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

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

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

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

    11 条回复    2017-05-17 17:29:02 +08:00
    artandlol
        1
    artandlol  
       2017-05-17 14:27:55 +08:00 via Android   ❤️ 1
    5 月 20 号了,其实原作者也只是想多钓一个萌汉子
    MrFireAwayH
        2
    MrFireAwayH  
    OP
       2017-05-17 14:28:49 +08:00
    @artandlol #1 什么鬼啦!
    littleylv
        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
    这种给提交了吧 = =
    MrFireAwayH
        4
    MrFireAwayH  
    OP
       2017-05-17 14:46:19 +08:00
    @littleylv #3

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

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

    祝幸福
    binux
        9
    binux  
       2017-05-17 16:46:30 +08:00
    @littleylv #3 我觉得这个改动量,获得一个署名并没有什么问题吧。
    0915240
        10
    0915240  
       2017-05-17 17:07:18 +08:00
    @R18 #8 快♂感
    MrFireAwayH
        11
    MrFireAwayH  
    OP
       2017-05-17 17:29:02 +08:00
    @binux #9 谢谢跳蚤菊苣支持~
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   5130 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 26ms · UTC 01:12 · PVG 09:12 · LAX 17:12 · JFK 20:12
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.