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

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

  •  
  •   TomVista · 7 天前 · 10007 次点击

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

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

    第 1 条附言  ·  7 天前
    感谢大家认可 和 捧场,祝大家代码写的开心,

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

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

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