-
Notifications
You must be signed in to change notification settings - Fork 52
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
feat(plugin-shiki): add built-in twoslash support #348
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for vuepress-ecosystem ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for vuepress-ecosystem ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@pengzhanbo It seems that you already implement a good approach in your theme, please consider migrating the feature upstream. |
Pull Request Test Coverage Report for Build 13199274422Details
💛 - Coveralls |
I see that the current PR uses In the early stages of my theme, I also used I suggest using If |
I would prefer optional deps plan, and if the deps are too many, we can consider providing an extra package which has them as deps, so that users can install a single package (e.g.: |
No description provided.