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

给 spring 提了第一个 pr

  •  1
     
  •   rhacker1995 · 2022-12-13 10:56:52 +08:00 · 4073 次点击
    这是一个创建于 705 天前的主题,其中的信息可能已经有所发展或是发生改变。

    我在 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   ·   实用小工具   ·   6231 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 27ms · UTC 02:32 · PVG 10:32 · LAX 18:32 · JFK 21:32
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.