人生第一次(也是第一个)pr 被 merge, 哈哈哈, 开心

This topic created in 2020 days ago, the information mentioned may be changed or developed.

本来不想提的, 提了吧, 说不定也是被忽略.
但是, 还是觉得, 试试吧, 说不定被 merge 了呢.
试试吧.

果然, 不到一天, 就被作者 merge 了, 作者还是一个大神.

被大神说 thanks 的感觉

awesome!!!

希望大家勿以 patch 小而不为.

merge

说不定

试试

awesome

15 replies • 2021-04-23 17:36:43 +08:00

ampedee

Apr 22, 2021 via Android

🐮🐸

nguoidiqua

Apr 22, 2021

可以发出来看看

timothyye

Apr 22, 2021

啥项目？分享一下

FurN1

Apr 22, 2021

/t/768587 似曾相识

francisdu

Apr 22, 2021 via Android

😏开源一小步，老弟一大步，加油

raaaaaar

Apr 22, 2021 via Android

的确，我也修过好几个 typo

asanelder

Apr 22, 2021

@nguoidiqua #2
@timothyye #3 不值一提, 就是文档有误, 哈哈
@francisdu #5 感谢感谢
@raaaaaar #6
@ampedee #1 哈哈

verzqli

Apr 23, 2021

好歹把项目发出来康康你改了啥啊

yangyaofei

Apr 23, 2021 via Android

给大项目提 pr 还是很有用的，过程中学了好多，包括测试用例等等的一对东西，就是慢…，提了以后几天后反馈，改了之后又几天，中间还会没人理几周，最终 merge 了都一个多月了……

fim

Apr 23, 2021

https://github.com/yurishkuro/opentracing-tutorial/pull/75 是这个吗

asanelder

Apr 23, 2021

@fim #10 这也太吓人了吧, 老哥怎么找到的???

asanelder

Apr 23, 2021

@fim #10 太吓人了, 社会工程学啊!!!

asanelder

Apr 23, 2021

@fim #10

俺猜老哥是 google 了俺的名字, 找到了俺的 github, 然后就蛛丝马迹.

是不是, 是不是

fim

Apr 23, 2021

@asanelder 完全正确

asanelder

Apr 23, 2021

@fim #14 np