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

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

  •  
  •   TomVista · 196 天前 · 10849 次点击
    这是一个创建于 196 天前的主题,其中的信息可能已经有所发展或是发生改变。

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

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

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

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

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

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