大家好,我是松柏。
前两天在修 bug 的时候,写了个indexOf
的方法。
这个方法是用来获取一段文本中某个字符串第 n 次出现的索引,
由于第一次写这个方法的时候少考虑了一种边界条件,导致最后查出的数据有时候会不符合预期。
我处理完这个bug
之后,就提交了代码,如下图:
鱼皮哥在做code review
的时候告诉我这个是有现成方法的,如下图:
因为我自己手动写过这个方法,再加上注释写的比较清楚,
所以我很快发现了注释的表述是有问题的,如下图:
由于之前没参与过开源,所以我是在跟鱼皮哥同步确认之后,才给Hutool
提了个修正注释的pr
。
(甚至给开源项目提交代码的流程都是现搜的🐶)
然后,这个pr
很快就被合并了!如下图:
因为是第一次给开源项目提pr
,而且还是这么知名的项目,所以当时还是非常激动的,
也算给开源做贡献了,虽然只是一行注释,哈哈哈。
欢迎关注co松柏
公众号一起学习交流!