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

人生中的第一个 PR 被 merge 了

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

    前端选手,一直追随巨佬 Anthony Fu 的脚步。

    昨天找到一个 1.5k🌟(star)项目的 bug ,
    研究源码一下午后,贡献了一次 commit ,
    并被作者感谢和 merge ,单独 release 一个小版本。

    现在和 contributor 列表的 4 位巨佬并列显示了,能开心一整天。

    总的来说,这第一次 PR ,感觉就像初恋一样~

    100 条回复    2022-07-03 19:17:19 +08:00
    theprimone
        1
    theprimone  
       51 天前   ❤️ 7
    nb!
    sadfQED2
        2
    sadfQED2  
       51 天前 via Android   ❤️ 2
    哈哈哈,我曾经发现了 TensorFlow 的 bug ,也是一样的心情
    gimp
        3
    gimp  
       51 天前   ❤️ 1
    恭喜🎉
    cyrbuzz
        4
    cyrbuzz  
       51 天前   ❤️ 1
    恭喜。
    etby
        5
    etby  
       51 天前   ❤️ 1
    恭喜,我第一个 PR 合并之后也很高兴
    seaswalker
        6
    seaswalker  
       51 天前 via iPhone
    被合并过两次,阿里的一个和 apache shiro
    SbloodyS
        7
    SbloodyS  
       51 天前   ❤️ 1
    恭喜恭喜
    Morii
        8
    Morii  
       51 天前   ❤️ 1
    恭喜,为开源做出了贡献
    NPC666
        9
    NPC666  
       51 天前   ❤️ 1
    恭喜🎉
    BiChengfei
        10
    BiChengfei  
       51 天前   ❤️ 1
    牛牛牛
    noyidoit
        11
    noyidoit  
       51 天前
    还以为大家都是从 docs typo fix 开始 hhhh
    min
        12
    min  
       51 天前   ❤️ 1
    jetbrain license 申请起来
    PPing520
        13
    PPing520  
       51 天前   ❤️ 1
    恭喜恭喜,我第一次被合并也很激动
    cosmic
        14
    cosmic  
       51 天前   ❤️ 1
    congrats!
    TomVista
        15
    TomVista  
       51 天前   ❤️ 1
    点赞👍
    ws52001
        16
    ws52001  
       51 天前   ❤️ 1
    恭喜恭喜,确实是个值得高兴的事。。
    anguiao
        17
    anguiao  
       51 天前
    我也给他提过一次,虽然只改了一行,233
    WillBC
        18
    WillBC  
       51 天前   ❤️ 1
    祝君更进一步
    Caratpine
        19
    Caratpine  
       51 天前   ❤️ 1
    恭喜恭喜
    tyrone2333
        20
    tyrone2333  
       51 天前   ❤️ 1
    总之,牛逼👍
    kaka6
        21
    kaka6  
       51 天前   ❤️ 1
    恭喜恭喜
    quicksand
        22
    quicksand  
       51 天前   ❤️ 1
    恭喜恭喜
    sivacohan
        23
    sivacohan  
       51 天前 via iPhone   ❤️ 1
    厉害厉害,我人生第一个被 merge 的 PR ,应该是 saltstack 。
    abigeater
        24
    abigeater  
       51 天前   ❤️ 1
    恭喜恭喜,我第一次合并时还一直搜索几个提 PR 的流程确保自己符合规范(虽然后面合进去后有几个 BUG )
    liuw666
        25
    liuw666  
       51 天前 via iPhone   ❤️ 1
    🎉
    ZxBing0066
        26
    ZxBing0066  
       51 天前
    过年那段时间看了几天 vite build 的源码,提了 3 个 pr 修了 3 个 bug ,到现在居然还在榜上,#99 快被刷下去了 😂
    ryougifujino
        27
    ryougifujino  
       51 天前
    有幸给 Taro 提了几个 PR 也被合并了
    cszchen
        28
    cszchen  
       51 天前 via iPhone   ❤️ 1
    当年给 yii 提交了一个 pr ,合并以后我也像你一样兴奋
    Jooooooooo
        29
    Jooooooooo  
       51 天前   ❤️ 1
    恭喜!
    xiaomimei
        30
    xiaomimei  
       51 天前   ❤️ 1
    恭喜
    equationl
        31
    equationl  
       51 天前   ❤️ 2
    哈哈哈,菜鸡安卓,我的第一次是给一个有 17k star 的项目水了一个只有 4 行代码的 PR ,没想到居然被作者 merge 了。

    但是没有楼主厉害,能找到 BUG ,我只是加了一个理应存在,但是居然没有的特性,而且其实这个项目已经具备添加这个特性的一切条件了,但是它就是没加,所以我给加上了。
    anan1314git
        32
    anan1314git  
       51 天前   ❤️ 1
    优秀优秀
    bytesfold
        33
    bytesfold  
       51 天前   ❤️ 1
    恭喜,我提了个 PR 那人直接关了,确实是有问题,现在也懒得看了
    bytesfold
        34
    bytesfold  
       51 天前
    @bytesfold 确实是原仓库代码有问题,我的 PR 有没有问题不知道
    TateLiao
        35
    TateLiao  
       51 天前   ❤️ 1
    恭喜恭喜
    SeanTheSheep
        36
    SeanTheSheep  
       51 天前   ❤️ 1
    牛逼兄弟,恭喜恭喜
    qianxi0410
        37
    qianxi0410  
       51 天前   ❤️ 1
    恭喜!
    bruce0
        38
    bruce0  
       51 天前   ❤️ 1
    牛逼牛逼
    killva4624
        39
    killva4624  
       51 天前   ❤️ 1
    兄弟牛逼,我的项目被 ISSUE 和 FORK 都已经很开心了😄
    xta
        40
    xta  
       51 天前
    👍
    moioooo
        41
    moioooo  
       51 天前
    曾经私聊一个库的作者,说 README.md 里有错别字。
    然后让我提 PR 。
    我就提了,被合并进去了。
    脑子嗡嗡的。
    frostnotfall
        42
    frostnotfall  
       51 天前 via Android   ❤️ 1
    恭喜恭喜,欢迎为开源做贡献,我第一个被 merge 的是 youtube-dl ,也是很开心
    ccyu220
        43
    ccyu220  
       51 天前
    我的第一个 PR 是解决报错问题,虽然并不会影响项目使用,哈哈哈哈
    luffy
        44
    luffy  
       51 天前   ❤️ 1
    恭喜,

    虽然我有 n 个 pr 已经被 merge ,但还是忘不了第一个
    so2back
        45
    so2back  
       51 天前   ❤️ 1
    恭喜
    frostnotfall
        46
    frostnotfall  
       51 天前 via Android
    @frostnotfall 不对,这是第二个,第一个被 merge 的项目,当时还带了无用的历史代码上去了
    NeezerGu
        47
    NeezerGu  
       51 天前
    想当初我看到 tf 的一行文本错误,果断提交了 pr (也是人生第一次),好像需要 3 个人通过,结果前两个通过了,最后一个没通过。。
    现在也不知道该在哪里看了……
    cassyfar
        48
    cassyfar  
       51 天前   ❤️ 1
    congrat!
    monkeyWie
        49
    monkeyWie  
       51 天前   ❤️ 1
    哈哈 恭喜恭喜,给阿里提交两个 pr 的飘过
    ddzzhen
        50
    ddzzhen  
       51 天前 via Android   ❤️ 1
    恭喜恭喜
    wukongkong
        51
    wukongkong  
       51 天前 via Android
    uniapp 三四次了,很开心😁
    zhanggang807
        52
    zhanggang807  
       51 天前   ❤️ 2
    想起当年我的第一个 pr 被 mr ,然后 app 关于界面贡献者里出现了我的账号,那个开心啊
    比完成公司的一个项目都开心
    LuciferCLC
        53
    LuciferCLC  
       51 天前
    cool~
    amwyyyy
        54
    amwyyyy  
       51 天前
    给 skywalking 提过两个,还收到了前 500 贡献者的证书
    Merlini
        55
    Merlini  
       51 天前
    我最激动的一次是给 huggingface 的 transformers 修了个小 bug 非常开心
    git00ll
        56
    git00ll  
       51 天前   ❤️ 1
    研究了三天,一次给 openfeign 提交一次 pr ,当时也特别开心
    dolphintwo
        57
    dolphintwo  
       51 天前
    恭喜!!
    SimbaPeng
        58
    SimbaPeng  
       51 天前
    恭喜恭喜
    bjfane
        59
    bjfane  
       51 天前
    @noyidoit 恭喜楼主,就服你。
    misaka19000
        60
    misaka19000  
       51 天前
    恭喜💐
    anoninz
        61
    anoninz  
       51 天前
    恭喜!🎉
    XCFOX
        62
    XCFOX  
       51 天前
    🎉🎉🎉
    youngster
        63
    youngster  
       51 天前
    恭喜
    demoshengxw
        64
    demoshengxw  
       51 天前 via iPhone
    @seaswalker 在这里第一看见我抬的球迷😂
    EminemW
        65
    EminemW  
       51 天前
    恭喜恭喜,我昨天也成功被合并了一次 PR
    bigxianyu
        66
    bigxianyu  
       51 天前 via Android
    @min 有 pr 就可以吗
    gamexg
        67
    gamexg  
       51 天前
    恭喜!
    realpg
        68
    realpg  
       51 天前
    我的 github 号多到都快记不住了……
    尤其是常年混匿名社交,还有配套匿名 github 号
    closedevice
        69
    closedevice  
       51 天前
    从来发现不了问题咋办
    freshmanc
        70
    freshmanc  
       51 天前
    😂 我第一而且唯一的一个是修 README 的 typo
    christin
        71
    christin  
       51 天前 via iPhone
    牛逼 恭喜!
    chuanqirenwu
        72
    chuanqirenwu  
       51 天前
    牛逼,感谢对开源社区的贡献。
    DT27
        73
    DT27  
       51 天前
    开源就是好,人多力量大。
    Moha
        74
    Moha  
       51 天前
    恭喜 !
    dengqing
        75
    dengqing  
       51 天前 via iPhone
    一样的心情!后面一发不可收拾,提了百来个 PR 了
    redford42
        76
    redford42  
       51 天前
    好棒!
    Bingchunmoli
        77
    Bingchunmoli  
       51 天前
    知道 spring 的 bug 但是不会解决,只能提 ISS 的菜逼路过
    FrankAdler
        78
    FrankAdler  
       51 天前 via iPhone
    去年给两个开源项目提了 pr ,第一个比较松,解决问题就给合并了,第二个是国外的项目,负责人比较在意流程和规范以及代码质量(比如提交颗粒拆小点,换行,本次提交是上一个的修复要使用 fixup ,向下兼容说明等等),各种交涉调整提交,最后终于合并了,感觉学到很多然后觉得自己污染了前面一个项目的 commit log ,但是为时已晚😂
    ngn999
        79
    ngn999  
       51 天前   ❤️ 1
    大家可以用这个 https://firstpr.me 去看看自己的第一个 pr
    adian
        80
    adian  
       51 天前
    牛逼
    SHF
        81
    SHF  
       51 天前   ❤️ 3
    Kaiv2
        82
    Kaiv2  
       51 天前
    👍
    g190875565
        83
    g190875565  
       51 天前
    恭喜,我前两天也第一次 PR 被 merge ,然后去别的项目又提了几个
    imaginistx12
        84
    imaginistx12  
       51 天前 via Android
    👏
    LucasWang
        85
    LucasWang  
       51 天前 via Android
    恭喜恭喜
    linyi01
        86
    linyi01  
       51 天前
    恭喜老哥!!!
    fengxiu
        87
    fengxiu  
       51 天前
    大哥 太强了 开源项目需要你这样有“找茬”的人才👍
    Dragonphy
        88
    Dragonphy  
       51 天前
    我的第一个 PR 是修个运行错误😂,那个作者也在 V 站
    bug4c
        89
    bug4c  
       51 天前 via Android
    恭喜🎉
    xrr2016
        90
    xrr2016  
       51 天前
    恭喜恭喜🎉
    MaxFang
        91
    MaxFang  
       51 天前
    感觉大家都好厉害啊,目前只给几个小项目提过 pr ,还需要努力。
    owlbatex
        92
    owlbatex  
       51 天前
    恭喜
    LeslieLeung
        93
    LeslieLeung  
       51 天前
    有幸给 uptime-kuma 提过 pr 并被 merge ,当时在用企业微信做通知渠道发现它没支持,依样画葫芦架上了企业微信的通知渠道( uptime-kuma 是 js 项目,我是搞后端的),兴奋了好久
    rpman
        94
    rpman  
       51 天前
    这个项目很多 bug, 欢迎提 PR 修 🐶
    https://github.com/logseq/logseq/issues
    Pogbag
        95
    Pogbag  
       50 天前 via iPhone
    开源项目的伟大之处
    exploreexe
        96
    exploreexe  
       50 天前
    牛逼!
    LudwigWS
        97
    LudwigWS  
       50 天前 via iPhone
    @rpman 不行,怕累着维护者,review 不过来怎么办🐶
    cloudzhou
        98
    cloudzhou  
       50 天前
    RiceMarch
        99
    RiceMarch  
       50 天前
    @noyidoit 哈哈哈哈哈哈哈哈哈哈 心声!
    iceworld724
        100
    iceworld724  
       42 天前
    程序员的快乐真简单
    关于   ·   帮助文档   ·   API   ·   FAQ   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   1011 人在线   最高记录 5497   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 46ms · UTC 21:03 · PVG 05:03 · LAX 14:03 · JFK 17:03
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.