Skip to content

Conversation

@NoiseFan
Copy link
Collaborator

@NoiseFan NoiseFan commented Oct 28, 2025

@netlify
Copy link

netlify bot commented Oct 28, 2025

Deploy Preview for cn-vitest ready!

Name Link
🔨 Latest commit d0dca39
🔍 Latest deploy log https://app.netlify.com/projects/cn-vitest/deploys/6905f9225a00ad0008288e53
😎 Deploy Preview https://deploy-preview-742--cn-vitest.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@thinkasany thinkasany requested a review from lxKylin November 1, 2025 06:08
@thinkasany
Copy link
Collaborator

解决一下冲突,这个 PR 怎么样啦,要不然先合一遍? 不然下次同步上游分支之后又会带来巨额冲突
image

@NoiseFan
Copy link
Collaborator Author

NoiseFan commented Nov 1, 2025

对于这个任务,我本地写了一些e2e的测试。我要检查完毕,在准备合并。

@NoiseFan
Copy link
Collaborator Author

NoiseFan commented Nov 1, 2025

我来继续解决吧。看起来越来越多了!

@NoiseFan
Copy link
Collaborator Author

NoiseFan commented Nov 1, 2025

@thinkasany 佬,帮忙合并一次吧。确实解决冲突比较繁琐。

@NoiseFan NoiseFan marked this pull request as ready for review November 1, 2025 12:14
@thinkasany thinkasany merged commit ad73ea0 into vitest-dev:dev Nov 1, 2025
4 checks passed
@thinkasany
Copy link
Collaborator

感谢你的贡献,辛苦啦~ Thanks~

@thinkasany
Copy link
Collaborator

image 是不是 rebase 之后没有 push -f,不过没关系,用了 squash merge

@thinkasany
Copy link
Collaborator

@thinkasany 佬,帮忙合并一次吧。确实解决冲突比较繁琐。

主要是这个下次机器人同步的时候会带来比较大的冲突,得看你那边好不好解决了,因为和今天合的冲突太多了,就先关了,可以等机器人同步过后再做比较大的 PR,现在做点小的,拆开及时合掉就行啦。
image

@thinkasany
Copy link
Collaborator

对于这个任务,我本地写了一些e2e的测试。我要检查完毕,在准备合并。

怎么样的 e2e,解决了什么问题,是否可以提交到仓库呢?

@NoiseFan
Copy link
Collaborator Author

NoiseFan commented Nov 1, 2025

检查当前中文文档的所有标题的自定义锚点和英文是否一致
这个e2e是使用Playwright写的
修改Markdown 还是手动改,并没有完成自动化的。
到是可以考虑写一些eslint,自动处理这些事情。
但是这些eslint 可能仅适用于当前文档库,不知道你怎么认为?

@thinkasany
Copy link
Collaborator

检查当前中文文档的所有标题的自定义锚点和英文是否一致 这个e2e是使用Playwright写的 修改Markdown 还是手动改,并没有完成自动化的。 到是可以考虑写一些eslint,自动处理这些事情。 但是这些eslint 可能仅适用于当前文档库,不知道你怎么认为?

倒是可以提交个 PR,我拉下来体验一下~ 仅检查当前库也很好用了,后续也可以 来 vite 中文站点试试,用场景慢慢丰富功能可能就变成一个好用的库了

@NoiseFan
Copy link
Collaborator Author

NoiseFan commented Nov 1, 2025

仓库地址:https://github.com/NoiseFan/e2e-vitest-docs
现在还是在面向任务去写,而且写的比较潦草,还有bug。但是大部分页面还是可以跑的通的

可以按照这个截图跑一下试试
image

@thinkasany
Copy link
Collaborator

你这个东西做出来还是蛮有用的,目前 vitest / vite 中文站都是我在维护的,你这个还能够做检查,可以发现一些文档的错误,做的好的话是可以慢慢推起来的。

但是 eslint 没办法检查链接是否有效吧,看来是做静态的格式检查。

不过 e2e 的话,倒是可以配置到 ci 里面自动运行,然后可以通过一些 node 脚本结合网络状态判断锚点是否对的。

roi 似乎不是很够,个人有兴趣的话,可以了解一下~

我觉得你可以试试,也欢迎来 antd 的仓库尝试一下,那边也有锚点,如果能查出一些问题,还能来 PR,场景多了应该能越做越好。

@NoiseFan
Copy link
Collaborator Author

NoiseFan commented Nov 1, 2025

eslint仅在编辑中能起到一些效果。收益其实不是特别大。
目前这个纯是自己为了更快速解决问题,写出的初步的东西。

另外,roi 这个名词讲的是投资回报率吗?我没太理解你的意思。

vitest这边文档处理干净了,到是可以试试viteantd

@thinkasany
Copy link
Collaborator

thinkasany commented Nov 1, 2025

eslint仅在编辑中能起到一些效果。收益其实不是特别大。 目前这个纯是自己为了更快速解决问题,写出的初步的东西。

另外,roi 这个名词讲的是投资回报率吗?我没太理解你的意思。

vitest这边文档处理干净了,到是可以试试viteantd

就是投下去的时间可能不小,但是最后产生的价值可能没那么大,但是有肯定是有的,说不定推的仓库多了就完善起来了

@NoiseFan NoiseFan deleted the docs/anchor branch November 1, 2025 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants