第一次给开源项目做贡献,我给 Hutool 改了行注释🎉

25 阅读1分钟

大家好,我是松柏。

前两天在修 bug 的时候,写了个indexOf的方法。

这个方法是用来获取一段文本中某个字符串第 n 次出现的索引,

由于第一次写这个方法的时候少考虑了一种边界条件,导致最后查出的数据有时候会不符合预期。

我处理完这个bug之后,就提交了代码,如下图:

image.png

鱼皮哥在做code review的时候告诉我这个是有现成方法的,如下图:
企业微信截图_140326b0-c80e-44c8-b777-7cbc1196d835.png

因为我自己手动写过这个方法,再加上注释写的比较清楚,

所以我很快发现了注释的表述是有问题的,如下图:
企业微信截图_3e415d14-dfa9-4d58-8c6b-3f586033576c.png

由于之前没参与过开源,所以我是在跟鱼皮哥同步确认之后,才给Hutool提了个修正注释的pr

(甚至给开源项目提交代码的流程都是现搜的🐶)

然后,这个pr很快就被合并了!如下图:
image.png

因为是第一次给开源项目提pr,而且还是这么知名的项目,所以当时还是非常激动的,

也算给开源做贡献了,虽然只是一行注释,哈哈哈。

欢迎关注co松柏公众号一起学习交流!