嘻嘻,给 vueuse 的一个 pr 被 merge 啦我像麋鹿一样在林荫中走着,为着自己的香气而发狂;夜晚是五月正中的
我像麋鹿一样在林荫中走着,为着自己的香气而发狂;夜晚是五月正中的夜晚,清风是南国的清风; 我迷路了,我游荡着,我寻求那得不到的东西,我得到了我所没有寻求的东西。 —— 泰戈尔
感谢
首先感谢下大佬:Alfred-Skyblue! 正如标题所言,我给 vueuse
项目第一次提交的一个pr
被antfu
佬合并,之所以感谢佬,是因为大佬给我很多指导,就差手摸手教学了🤪,非常有耐心的佬了,有问必答,哈哈哈。
pr的契机
因为佬是 vueuse
的成员,然后佬自己维护了一套中文文档,我有幸pr了几个,但是还是得不到满足,可能是 star 少的原因吧(狗头保命),然后中午下班无意间打开 vueuse 看到 有个 issues
大概意思就是说需要返回个 id
吧,然后抱着好奇(万一混个pr呢)的心理,打开了源码,看了下,就是把 EventSource
封装了下,然后打开了 MDN查看了下文档,确实在 event
上有个lastEventId
属性,并没有被返回,关于lastEventId
的解释为:表示事件源的最后一个事件 ID。
pr过程
搞清楚后,于是,我就写出来这样的代码
是的,你没有看错,我新加了一个 id 变量,以及用这个 id 赋值等,return也是这个 id,当我提交后,大佬看了我的代码,摇摇头说,既然它内部都是lastEventId
你为什么要改成 id 呢,这样容易引起歧义,于是,大佬帮我 review
了一次
自此 pr 就完成了。但是当我并没有抱有任何希望被 merge 的时候,过了一周后,也就是今晚,大佬告诉我,pr被 merge 了。
附上 merge 截图
结论
今天心情很好,记录记录,最后欢迎大佬们都加入开源中来!用爱发电!
转载自:https://juejin.cn/post/7373529708263129098