-
-
Notifications
You must be signed in to change notification settings - Fork 390
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
fix: restore missing langs/*.mjs exports #882
Conversation
Fixes the breaking change in 1.25
✅ Deploy Preview for shiki-matsu ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for shiki-next ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for shiki-next ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
No idea why the CI is failing, most likely not related. |
Hey @antfu 🙌 . Sorry to ping you directly but could you take a quick look 🙏 . We have a bunch of users not using the pinned version 😢 and have broken builds now. |
Which version are you using? 1.25.1? What error are you getting? It seems to work fine on my side. I don't think we touched the exports part |
Please provide a minimal reproduction. Thanks. |
Here is the repro: https://github.com/RomanHotsiy/shiki-bug-repro |
UPDATE. It seems to be fixed on 1.26.1 in this commit: 5766b28 Thanks! Closing. |
Description
Fixes the breaking change in 1.25 related to
shiki/langs/<lang>.mjs
importsLinked Issues
FIxes #881
Additional context