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

给 spring 提了第一个 pr

  •  1
     a href="Javascript:" Onclick="downVoteTopic(902126);" class="vote">
  •   rhacker1995 2022-12-13 10:56:52 +08:00 4571 次点击
    这是一个创建于 1035 天前的主题,其中的信息可能已经有所发展或是发生改变。

    我在 github 上给 spring 提了个 pr ,spring 没有直接合并,可能觉得我的单元测试不够,但他们在我的 commit 上补充了单元测试,直接 push 到 master 上,所以还是保留了我的 commit 信息的。 这种情况下我能吹嘘自己是 contributor 吗,能把这写进简历吗 pr 地址 https://github.com/spring-projects/spring-framework/pull/29631

    15 条回复    2022-12-13 15:22:16 +08:00
    serco
        1
    serco  
       2022-12-13 11:00:28 +08:00
    有啥不可以,很多只给文档修过 typo 的都可以称作 contributor
    alexsunxl
        2
    alexsunxl  
       2022-12-13 11:19:08 +08:00
    不用把 pr 写进去,直接说是贡献者就行了。一个被 close 的 pr ,容易被较真。
    真要贴链接,把带你名字账号的 commit 贴进去呗
    wuwei12138
        3
    wuwei12138  
       2022-12-13 11:35:14 +08:00
    这个问题是不是在 maimai 上面看到过。。
    me221
        4
    me221  
       2022-12-13 11:41:50 +08:00   2
    小公司不懂, 大公司看不上...
    Dxxxxs
        5
    Dxxxxs  
       2022-12-13 11:46:54 +08:00
    昨天才在 maimai 上刷到过一样的帖子
    Sasasu
        6
    Sasasu  
       2022-12-13 11:52:13 +08:00
    他这是嫌弃你有两个 commit ,你应该 rebase -i push -f 合并一下
    chendy
        7
    chendy  
       2022-12-13 12:06:28 +08:00
    算(来自给 spring 修过好几个文档 typo 的文档翻译人…
    littlecreek
        8
    littlecreek  
       2022-12-13 12:08:45 +08:00
    这当然算, 你有什么理由怀疑自己不是?
    alexsunxl
        9
    alexsunxl  
       2022-12-13 12:18:22 +08:00
    @me221 别这么二极管呀。 那不是还有很多中间的嘛
    yangyaofei
        10
    yangyaofei  
       2022-12-13 12:39:51 +08:00 via Android
    spring 在 github 上貌似不是主 repo 大部分 pr 都是关闭的,你看看别的 pr 也是这样的
    Macolor21
        11
    Macolor21  
       2022-12-13 12:42:27 +08:00
    Spring 这边的 PR 似乎都是这么处理的(我的代码提交也是),对新人完全不友好。(可能是那种乱提文字纠错让自己看起来贡献很大的人太多了,印度,中国都特别多这种)



    @Sasasu #6 和 commit 数量无关,merge 的时候 maintainer 可以选择是否 squash commit 的
    hello158
        12
    hello158  
       2022-12-13 14:07:12 +08:00
    这有啥可问?
    你要觉得这个能体现你的水平,是可以吹嘘的水准,你就写呗?
    Sasasu
        13
    Sasasu  
       2022-12-13 14:10:56 +08:00
    @Macolor21 squash 了你的签名就没了
    tutu2000
        14
    tutu2000  
       2022-12-13 15:21:42 +08:00
    @hello158 不算吹嘘,这就是 contributor
    wbd31
        15
    wbd31  
       2022-12-13 15:22:16 +08:00
    https://github.com/spring-projects/spring-framework/commit/a7bf14b3641e9147881336a81bae5b1008db29b8

    CoderYellow authored and rstoyanchev committed 8 days ago

    重新提交后,原作者的 authored 信息是怎么保留的?
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     5802 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 24ms UTC 06:31 PVG 14:31 LAX 23:31 JFK 02:31
    Do have faith in what you're doing.
    ubao snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86