再开学大四了,给vuex提交个pr,被merge了,爽!

531 阅读3分钟

一、前言

hello,大家好,我是gsc,目前是一个开学即将大四的学生。

今天因为机缘巧合吧,发现了一个bug,给vuex提了个pr,然后被merge了。

下面给大家稍微聊聊个人的心路历程,这也是我在掘金发的第一篇文章。

我的github链接,欢迎关注:github.com/heygsc

pr链接:github.com/vuejs/vuex/…

二、发现bug

今天在公司的时候,刚午休完事,突然想起来之前面试官问过的一个问题:你了解vuex吗?

各位前端大佬想必都是vuex以及pinia的专家,对这个问题应该信手拈来。

然后想着,看看vuex的文档长啥样,估计也是用的vuepress。

用前阵子很流行的话说,看了两眼vuex文档,命运的齿轮开始转动。

大家看文档可能注意过,下面有时候会有一个编辑链接,点开可能是文档的源码,可以进行编辑。

当然了,真的编辑可能需要提pr什么的,不过链接也可以方便地查到文档的源码。

image.png

可能是我对开源比较感兴趣吧,想着随便点一下看看,估计和别的文档一样。

image.png

好,点开直接404了。

三、修复

正常来说是可以点开的,可能会涉及权限问题,也不应该404啊。

直觉告诉我,对,这个可能是个bug。

因为之前没怎么用过vuepress,这里说个题外话,之前用mkdocs(也大概是文档框架)做过一个有意思的段子集合,欢迎来看:github.com/heygsc/xiao…

把仓库弄到本地,搜索下编辑此页,发现vuex用到的是editLinkText

想稍微吐槽下,github的在线搜索只能搜文件名,好像搜不了代码内容🤣

然后各位请看vuepress官方文档中,editLinkText附近的editLinkPattern,或许是用来配置,这个编辑页面的链接的。

要解决了是吧。

嘿,您猜怎么着,在vuex中,我发现,貌似,并没有用这个editLinkPattern

但是vuepress文档告诉我,不设置这个的话,可以用docsRepo。好吧,这倒正常。

嘿,您再猜下怎么着,这下肯定能猜对了,是的,vuex文档也没用这个docsRepo

没事,这倒正常,因为vuepress文档还说了,也不用这个的话,默认用repo😄

到了repo发现,好吧,之前走弯路了,没学过vuepress,无所谓,好在找到了。

vuex的文档中,把repodocsDir,docsBranch放到一起,这几个就是vuex文档中,编辑此页指向的链接url,的重要组成部分

为什么404呢?

嘿嘿,它的分支名叫4.0,vuex中目前看不到这个分支❤️

我猜测,这个可能是之前文档配置的老哥失误,也可能是曾经存在这个分支,后来因为各种原因这个分支被删去了。这个都不好说。

现在就很好说了,分支改成main,结束。

四、merge

各种原因吧,pinia等的盛行,vuex貌似维护的频率不高了,我的这个commit,好像是vuex今年第二个commit✌️

image.png

这里放一下给我merge的老哥的回复

image.png

啊,看到github发来的merge邮件,和这个老哥的回复,我突然间恍惚了。

白天发现bug,提交pr,一直担心不会被merge。

倒不是担心不值得修复,我很自信的是,这确实是个问题,并且应该可以修复成功。

只是从commit的频率来看,近期pr数量本就不多,之前可以看到的今年唯一的commit,还是尤雨溪的。

果然啊,他们还记得vuex。

vuex啊,人们还记得你,还惦记着你,物是人非,人来人往,vuex还是会merge pr的。

许多年后,可能仍然会听到有人说:“你听说过vuex吗?那是我接触的第一个,专门为vue开发的库。”