-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
docs: apply twoslash #4560
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
docs: apply twoslash #4560
Conversation
…ilities` section
@serkodev You need to change the node Just try with 8192 (the free tier is limited to 6GB), iirc Anthony doesn't have any restriction at Netlifly |
@userquin Oh! Thanks for the reminder. I’m currently researching whether there’s a way to reduce build memory usage (it does require more than 16GB at the moment). So, it’s still uncertain whether this flag will be changed. If I decide to modify it later, I’ll add it to Netlify’s config. 👍 |
@serkodev don't waste your time, looks like it is a problem with Vue version, check:
If we can run the build at Netlify with 16GB it should be fine /cc @antfu In the meantime you can also try using |
@serkodev Maybe you can try with VP 2.0.2.alpha (using Vite 6) in a new local branch from this one (or even using a new clean clone of the branch, no idea if updating to Vite 6 will have some side effect on the build): https://github.com/vuejs/vitepress/releases (If you don't have time, I can give it a try next Saturday) |
we can also try updating the dependency to latest version 2.3.2: https://github.com/vueuse/vueuse/blob/main/pnpm-workspace.yaml#L12 |
Thx @brc-dd :
|
@userquin Thanks for providing this information and saving me a lot of time! 💚 I’ll take a look and see if it fixes the issue. |
Manually added `vite-plugin-pwa@^1.0.2` to satisfy peer dependency required by `@vite-pwa/vitepress@1.0.0`.
Looks like it works 👀 |
Yep, after plenty of trial-and-error, I finally got it building successfully on Netlify! 😭 |
Currently I'm working on passing the |
Before submitting the PR, please make sure you do the following
fixes #123
).Description
Related: #4548 (comment)
This PR applies Twoslash to document of VueUse and still in progress.
ts
andvue
code blocks in theindex.md
document automatically apply Twoslash.vue
in all code blocks (silently).vue
imports while others do not. Instead of enforcing uniform importing or non-importing, I believe that keeping shorter code snippets import-free can enhance readability. Therefore, I aim to preserve the original code snippets as much as possible.noErrors
option by default.vitepress > vite
tonpm:rolldown-vite@latest
to reduce the memory usage and build time on Netlify.Preview
Progress
Core
Add-ons