给 spring 提了第一个 pr

2022-12-13 10:56:52 +08:00
 rhacker1995

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

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



@Sasasu #6 和 commit 数量无关,merge 的时候 maintainer 可以选择是否 squash commit 的
hello158
2022-12-13 14:07:12 +08:00
这有啥可问?
你要觉得这个能体现你的水平,是可以吹嘘的水准,你就写呗?
Sasasu
2022-12-13 14:10:56 +08:00
@Macolor21 squash 了你的签名就没了
tutu2000
2022-12-13 15:21:42 +08:00
@hello158 不算吹嘘,这就是 contributor
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 信息是怎么保留的?

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/902126

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX