V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
TomVista
V2EX  ›  程序员

被合并了人生第一个 pr,很开心

  •  
  •   TomVista · 2021-04-07 09:49:21 +08:00 · 12334 次点击
    这是一个创建于 1327 天前的主题,其中的信息可能已经有所发展或是发生改变。

    一共三几行,添加了一个简单的分支功能,

    算了不解释了,没错,我是来炫耀的,膨胀.JPEG

    第 1 条附言  ·  2021-04-07 11:43:46 +08:00
    感谢大家认可 和 捧场,祝大家代码写的开心,

    下沉了
    79 条回复    2021-04-08 18:49:55 +08:00
    adeng
        1
    adeng  
       2021-04-07 09:51:56 +08:00 via Android   ❤️ 2
    我还没过呢,等我也有了第一个,我也要膨胀.jpg
    CallMeReznov
        2
    CallMeReznov  
       2021-04-07 09:54:17 +08:00   ❤️ 1
    之前在 centos 上发现 ch340 的驱动有问题,然后去 github 上找了一个修改版编译替换成功.
    就只是添加了个 Test List 就很开心了
    cjpjxjx
        3
    cjpjxjx  
       2021-04-07 09:55:13 +08:00 via iPhone   ❤️ 2
    第一次 GitHub 提交就被合并了,Lean 大佬的 OpenWrt 项目
    paradoxie
        4
    paradoxie  
       2021-04-07 10:00:02 +08:00   ❤️ 1
    第一次通过了别人的一个 pr 的时候,当时也挺开心。。。
    crclz
        5
    crclz  
       2021-04-07 10:00:03 +08:00   ❤️ 1
    发现某个 JWT 库因为 Json 解析的逻辑在某些情况下会导致安全隐患,然后在文档中添加了警告与例子,PR 被接受了。
    95276
        6
    95276  
       2021-04-07 10:09:04 +08:00   ❤️ 1
    第一次被合并我只加了一行代码. 当时也很开心.
    etby
        7
    etby  
       2021-04-07 10:12:30 +08:00   ❤️ 2
    我当时也很开心,现在也会时不时想起来
    learningman
        8
    learningman  
       2021-04-07 10:14:56 +08:00 via Android   ❤️ 2
    我第一个 PR 好像是 typo 来着。。。
    thet
        9
    thet  
       2021-04-07 10:15:04 +08:00   ❤️ 1
    收到第一次 issue 也挺开心的
    andj4cn
        10
    andj4cn  
       2021-04-07 10:15:35 +08:00   ❤️ 1
    年初在家无聊给一个数据库找了个 bug 打了 fix,一番扯皮近期合并了,也很开心
    cat007
        11
    cat007  
       2021-04-07 10:16:57 +08:00   ❤️ 1
    第一次 PR 是改了个 typo,兴奋一整天
    th00000
        12
    th00000  
       2021-04-07 10:18:03 +08:00   ❤️ 1
    第一次给公共仓库提 PR 是 Vertx 项目, 然后被挂到官网贡献者南墙上了, 开心了好几天。
    darer
        13
    darer  
       2021-04-07 10:18:56 +08:00   ❤️ 1
    第一次被合并了比较开心是修了 VSFilterMod 的色彩矩阵的问题的一个 PR
    有种被认同了的感觉
    deepkolos
        14
    deepkolos  
       2021-04-07 10:22:12 +08:00   ❤️ 1
    给 three 提过 PR,进了 r127,关于 TAGLoader 的简单的优化
    fkdtz
        15
    fkdtz  
       2021-04-07 10:27:59 +08:00   ❤️ 2
    开源的魅力
    qW7bo2FbzbC0
        16
    qW7bo2FbzbC0  
       2021-04-07 10:28:01 +08:00   ❤️ 1
    bottle.py 提的 pr 前几天被 merge 了,给 config-parser 提的 issue 被解决了。其他的基本是被驳回或者无视
    Caratpine
        17
    Caratpine  
       2021-04-07 10:29:13 +08:00   ❤️ 1
    恭喜。我当年也是,还发了条朋友圈嘚瑟了一下,哈哈
    AoEiuV020
        18
    AoEiuV020  
       2021-04-07 10:29:31 +08:00   ❤️ 1
    网上的小透明突然在显眼的位置留下了名字,确实很开心,
    特地翻了下我提 pr 的历史,16 年起共被合并 6 条,
    最大的项目是 star 5.8k 的 jsonPath,
    johnnyNg
        19
    johnnyNg  
       2021-04-07 10:32:40 +08:00   ❤️ 1
    工作有一部分是开源项目,so
    chendy
        20
    chendy  
       2021-04-07 10:38:32 +08:00   ❤️ 1
    给 spring 修过好几次文档的格式拼写错误
    后来又一次 pr 被拒了,原因是检查了一下发现有一段文档问题太多他们自己重写了……
    ImACat
        21
    ImACat  
       2021-04-07 10:42:56 +08:00 via iPhone   ❤️ 1
    我的第一个 PR 是给某个包管理软件添加了一个新的软件
    ImACat
        22
    ImACat  
       2021-04-07 10:43:11 +08:00 via iPhone   ❤️ 1
    @ImACat 这个还是蛮容易的
    uiosun
        23
    uiosun  
       2021-04-07 10:43:31 +08:00   ❤️ 1
    祝贺!我第一个合并的时候,也很开心哈哈
    AoEiuV020
        24
    AoEiuV020  
       2021-04-07 10:43:59 +08:00   ❤️ 2
    找到楼主了,v2ex 好多人昵称都和 github 一样的,
    https://github.com/vueComponent/vue-jsx-hot-loader/pull/3/files
    lepig
        25
    lepig  
       2021-04-07 11:04:29 +08:00   ❤️ 2
    我的第一个 PR 是改了一个错别字。T.T
    snuglove
        26
    snuglove  
       2021-04-07 11:26:01 +08:00   ❤️ 2
    第一次 PR 改了一个双引号
    ysicing
        27
    ysicing  
       2021-04-07 11:39:24 +08:00   ❤️ 2
    第一次 pr 是 fix typo T.T
    zengzizhao
        28
    zengzizhao  
       2021-04-07 11:40:47 +08:00   ❤️ 1
    一样,第一次提交 pr 被合并的时候都很开心
    Sapp
        29
    Sapp  
       2021-04-07 11:43:04 +08:00
    第一次是给人修了一堆的 typescript 类型错误,没啥感觉,因为错误太多了
    Tumblr
        30
    Tumblr  
       2021-04-07 11:43:56 +08:00
    恭喜哦!
    除了那事儿第一次有点痛苦之外,貌似很多事第一次都是开心的。
    ayase252
        31
    ayase252  
       2021-04-07 11:44:13 +08:00 via iPhone
    第一次给 Node.js 写了一节文档😂
    Crossin
        32
    Crossin  
       2021-04-07 11:49:45 +08:00
    祝贺!
    zhongyx
        33
    zhongyx  
       2021-04-07 11:55:39 +08:00
    恭喜恭喜,羡慕了!
    774590465
        34
    774590465  
       2021-04-07 12:10:01 +08:00
    第一次 pr 是 disruptor,因为项目很老了,新加特性,后面又加了测试用例.前前后后好几个月才被合并.直接蹭到了 disruptor 前十的贡献者
    dongcidaci
        35
    dongcidaci  
       2021-04-07 12:48:46 +08:00
    羡慕大佬
    guanhui07
        36
    guanhui07  
       2021-04-07 12:49:34 +08:00 via Android
    恭喜恭喜🎉
    codeli
        37
    codeli  
       2021-04-07 13:00:57 +08:00
    羡慕大佬
    RayJiang9
        38
    RayJiang9  
       2021-04-07 15:09:14 +08:00
    第一次给别人提 PR 就被合并,和第一次别人给我提 PR,都挺开心的
    test005
        39
    test005  
       2021-04-07 15:09:51 +08:00   ❤️ 2
    我第一次 pr 被合并也挺高兴的,现在那项目已经有 28.6k star,贡献者们会被挂在 readme 和官网上。

    当时公司在基于那个项目做开发,我作为贡献者挂在官网上,期待着被同事发现,然后吹一波 nb,三年过去了。。。
    jixule
        40
    jixule  
       2021-04-07 15:14:27 +08:00
    我是在某个框架官网发现单词拼错了,提了 issue,作者改了也回了谢谢
    Akiya
        41
    Akiya  
       2021-04-07 15:38:24 +08:00
    我第一个 PR 也是修 Typo
    bigwhite
        42
    bigwhite  
       2021-04-07 16:48:37 +08:00
    非程序员,Adobe Premiere Pro ?
    MOONLIGHTT
        43
    MOONLIGHTT  
       2021-04-07 16:50:43 +08:00
    第一次 pr 是给 tensorflow 下面的一个仓库修 Type
    samasxxc
        44
    samasxxc  
       2021-04-07 17:02:34 +08:00
    我给 fastjson 提的代码也被合并了
    holycrazy
        45
    holycrazy  
       2021-04-07 17:02:36 +08:00
    羡慕中
    twor2
        46
    twor2  
       2021-04-07 17:25:18 +08:00
    这种凡体引起舒适
    Dashy
        47
    Dashy  
       2021-04-07 17:36:29 +08:00
    @test005 #39 红红火火恍恍惚惚或
    JustSong
        48
    JustSong  
       2021-04-07 17:42:57 +08:00
    第一次 PR 是给 Hexo 的 site 仓库添加自己写的主题
    solxnp
        49
    solxnp  
       2021-04-07 18:30:57 +08:00
    第一次 PR 是一个 bugfix (只有一行代码) 也很开心
    xiaket
        50
    xiaket  
       2021-04-07 18:47:55 +08:00
    第一次 PR 可能是在 Django, 那个时候 Django 都还在用 svn, 所以还是应该改称开源贡献...
    Jaeden
        51
    Jaeden  
       2021-04-07 18:57:22 +08:00
    我 pr 了翻译,给我回复说网站改版先不要翻译....
    zhangshine
        52
    zhangshine  
       2021-04-07 19:57:57 +08:00
    gitdoit
        53
    gitdoit  
       2021-04-07 20:13:22 +08:00
    第一次 PR 被拒了
    billwang
        54
    billwang  
       2021-04-07 20:18:31 +08:00
    加油!
    bigbigeggs
        55
    bigbigeggs  
       2021-04-07 20:44:54 +08:00
    给 dubbo 提了一个很贱的 pr,没想到竟然通过了
    bigbigeggs
        56
    bigbigeggs  
       2021-04-07 20:45:15 +08:00
    @bigbigeggs 很简单的 PR
    orzorzorzorz
        57
    orzorzorzorz  
       2021-04-07 22:24:27 +08:00
    加油啊。得趁着这段血还热乎的时候,能学多少是多少。
    毕竟之后,你就学惯了。:doge
    RadishWind
        58
    RadishWind  
       2021-04-08 08:23:44 +08:00
    在做毕设时,发现了 php debug 插件的一个 segment fault 提交后 owner 给合并了
    695975931
        59
    695975931  
       2021-04-08 08:37:39 +08:00
    恭喜啊~
    Originalee
        60
    Originalee  
       2021-04-08 08:45:04 +08:00
    typescript 文档最近更新了,提了一个 fix typo 被合并也很开心
    codingKingKong
        61
    codingKingKong  
       2021-04-08 09:19:46 +08:00
    只提过 issue 的发来贺电~
    chenzhenguo0620
        62
    chenzhenguo0620  
       2021-04-08 09:23:57 +08:00
    @Tumblr 开车!!
    XuSt
        63
    XuSt  
       2021-04-08 09:24:25 +08:00   ❤️ 2
    第一个 PR 是 Qt 源码,后来还被放到了官方 6.0 发布的时候的鸣谢名单里,开心哈哈哈哈
    JoStar
        64
    JoStar  
       2021-04-08 09:40:44 +08:00
    我也有几个 PR,不过都是些微不足道的工作。不过也算是个不错的开始
    cloudzhou
        65
    cloudzhou  
       2021-04-08 09:44:42 +08:00   ❤️ 1
    给 Golang 提交了一个,开心很久
    andytao
        66
    andytao  
       2021-04-08 10:08:51 +08:00
    人生第一次:作品和马化腾先生的产品并列在微软的软件仓库里,开心耶。。。
    https://www.v2ex.com/t/768668

    不知为何总是在列表中看不到,奇怪了
    fihserman123
        67
    fihserman123  
       2021-04-08 10:28:17 +08:00
    我第一次 pr 也是 Dubbo
    dvaknheo
        68
    dvaknheo  
       2021-04-08 10:55:04 +08:00
    我现在 提了个 pr,想再提交第二个 pr 不知道怎么处理。
    听说最快的方案是 删掉重新 fork,然后修改提交
    sonyxperia
        69
    sonyxperia  
       2021-04-08 11:12:11 +08:00 via Android
    别拿 typo 不当 pr
    dingyx99
        70
    dingyx99  
       2021-04-08 11:30:00 +08:00
    可喜可贺
    Dayz
        71
    Dayz  
       2021-04-08 12:24:54 +08:00
    @bigwhite pull request (“我改了你们的代码,你们拉回去看看”)
    kuro1
        72
    kuro1  
       2021-04-08 13:33:22 +08:00
    开源的魅力,懂的都懂哈哈
    574402766
        73
    574402766  
       2021-04-08 14:05:27 +08:00
    恭喜~
    wxsm
        74
    wxsm  
       2021-04-08 14:24:35 +08:00   ❤️ 1
    说来搞笑,有一段时间在学 k8s,发现中文文档好多问题,于是看到一处就用页面上的 edit this page 功能发起一个 PR,发起了十几个以后,那审核的哥们都有点烦了,说你能不能一次性来(我也很委屈啊,你们的这个功能就是发现一处就提一个 PR )。不过最后还是都被合并了。
    realpg
        75
    realpg  
       2021-04-08 14:32:38 +08:00
    我的第一次 merge 是给一个大项目改了描述。。。
    当然不是故意混 pr merge 次数的,是真写错了
    oIMOo
        76
    oIMOo  
       2021-04-08 14:45:07 +08:00
    话说怎么优雅的 PR?
    萌新表示,难道一定要需要:fork - modify - commit & push - PR?
    一定的话,PR 被合并之后,fork 到自己仓库的就能删了吧(删自己的影响原项目的 master 分支么)?
    AoEiuV020
        77
    AoEiuV020  
       2021-04-08 16:13:33 +08:00
    @oIMOo 打开别人项目里的文件,直接点铅笔图标编辑就能自动完成其他步骤了,
    HKEdward
        78
    HKEdward  
       2021-04-08 17:36:19 +08:00
    恭喜!看完回复,感觉又有动力写 bug 了。
    Sendya
        79
    Sendya  
       2021-04-08 18:49:55 +08:00
    ![image.png]( https://i.loli.net/2021/04/08/MjDYJL7UaGkrTti.png) 老哥可以呀,我都还没用上,你就用上啦
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   2619 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 30ms · UTC 04:39 · PVG 12:39 · LAX 20:39 · JFK 23:39
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.