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

2021-04-07 09:49:21 +08:00
 TomVista

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

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

12446 次点击
所在节点    程序员
79 条回复
codingKingKong
2021-04-08 09:19:46 +08:00
只提过 issue 的发来贺电~
chenzhenguo0620
2021-04-08 09:23:57 +08:00
@Tumblr 开车!!
XuSt
2021-04-08 09:24:25 +08:00
第一个 PR 是 Qt 源码,后来还被放到了官方 6.0 发布的时候的鸣谢名单里,开心哈哈哈哈
JoStar
2021-04-08 09:40:44 +08:00
我也有几个 PR,不过都是些微不足道的工作。不过也算是个不错的开始
cloudzhou
2021-04-08 09:44:42 +08:00
给 Golang 提交了一个,开心很久
andytao
2021-04-08 10:08:51 +08:00
人生第一次:作品和马化腾先生的产品并列在微软的软件仓库里,开心耶。。。
https://tanronggui.xyz/t/768668

不知为何总是在列表中看不到,奇怪了
fihserman123
2021-04-08 10:28:17 +08:00
我第一次 pr 也是 Dubbo
dvaknheo
2021-04-08 10:55:04 +08:00
我现在 提了个 pr,想再提交第二个 pr 不知道怎么处理。
听说最快的方案是 删掉重新 fork,然后修改提交
sonyxperia
2021-04-08 11:12:11 +08:00
别拿 typo 不当 pr
dingyx99
2021-04-08 11:30:00 +08:00
可喜可贺
Dayz
2021-04-08 12:24:54 +08:00
@bigwhite pull request (“我改了你们的代码,你们拉回去看看”)
kuro1
2021-04-08 13:33:22 +08:00
开源的魅力,懂的都懂哈哈
574402766
2021-04-08 14:05:27 +08:00
恭喜~
wxsm
2021-04-08 14:24:35 +08:00
说来搞笑,有一段时间在学 k8s,发现中文文档好多问题,于是看到一处就用页面上的 edit this page 功能发起一个 PR,发起了十几个以后,那审核的哥们都有点烦了,说你能不能一次性来(我也很委屈啊,你们的这个功能就是发现一处就提一个 PR )。不过最后还是都被合并了。
realpg
2021-04-08 14:32:38 +08:00
我的第一次 merge 是给一个大项目改了描述。。。
当然不是故意混 pr merge 次数的,是真写错了
oIMOo
2021-04-08 14:45:07 +08:00
话说怎么优雅的 PR?
萌新表示,难道一定要需要:fork - modify - commit & push - PR?
一定的话,PR 被合并之后,fork 到自己仓库的就能删了吧(删自己的影响原项目的 master 分支么)?
AoEiuV020
2021-04-08 16:13:33 +08:00
@oIMOo 打开别人项目里的文件,直接点铅笔图标编辑就能自动完成其他步骤了,
HKEdward
2021-04-08 17:36:19 +08:00
恭喜!看完回复,感觉又有动力写 bug 了。
Sendya
2021-04-08 18:49:55 +08:00
![image.png]( https://i.loli.net/2021/04/08/MjDYJL7UaGkrTti.png) 老哥可以呀,我都还没用上,你就用上啦

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

https://tanronggui.xyz/t/768587

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

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

© 2021 V2EX