-
-
Notifications
You must be signed in to change notification settings - Fork 382
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
Plan for 1.0 #424
Comments
You got this! |
Excited! |
I haven't been able to get Shiki working in Next.js when deployed to Vercel and I think a fix would fall under
You can see my struggles with trying to get it deployed to Vercel over in this repo: https://github.com/tom-sherman/shiki-server-components-demo It works great locally by the way, it's just deploying to vercel isn't compatible with the way shiki wants to read grammars and themes from the file system local to it's node_modules directory. |
Wouldn't it be reasonable to adjust the README?
vs. https://github.com/shikijs/shiki/releases/tag/v0.14.2 |
Wanted to give an update on my struggles, I was able to get it deployed to Vercel but I had to drop the shiki dependency :-/ In the end I used https://github.com/code-hike/lighter which feels like what Shiki 1.0 (or maybe 2.0?) could aim to be with regards to the API and the way things work. |
True, it is not working with dynamic server component in Next.js when deployed to Vercel |
We should replace this entire repo with @antfu's shikiji for 1.0, https://github.com/antfu/shikiji And get plugin support in, then we can bring shiki-twoslash to be a shikiji plugin |
The repo contains all the history, therefore it shouldn't be replaced. The other repo is only 3 weeks old, but I guess @antfu could create a PR to re-do his rewrite on a branch (hopefully only the initial setup, then manually merge-pick the revisions from his repo into this branch), if it is sure that his changes are accepted he may do that? |
We need consensus from pine |
So I made a few API design changes in If you want to move faster, I'd be happy to hear your feedback on https://github.com/antfu/shikiji so we could roll out features to experiments, and eventually port back to A side note that I also made a |
@orta I'd be happy to explore the plugin API design in shikiji that fits twoslash! Let me know if you have any initial ideas. |
Realistically I can't think about anything hard for a month or two, so consider it on my backlog 👍🏻 |
I think your technique for integration twoslash into shikiji natively looks great, I don't think we need to go back to that - but I am re-raising that I think shikiji should replace shiki! |
Any update on this |
Talked to @octref and we made the consensus to merge shikiji back to shiki, I'll take the lead and start the work soon 🎉 |
Well, I haven't been spending much time on OSS for a while, and now I plan to get back to it. So here's a plan for 1.0. I'm guessing it'll take 2 months. This is a brain dump list and will change as things goes.
Todo – Meta
dev
branchdev
back to main for releasing 1.0Todo – Open to contributions
The text was updated successfully, but these errors were encountered: