Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Migrate shiki-es dep to shikiji #76

Open
josecelano opened this issue May 30, 2024 · 2 comments
Open

Migrate shiki-es dep to shikiji #76

josecelano opened this issue May 30, 2024 · 2 comments
Assignees
@grmbyrn
Labels
Dependencies Related to Dependencies
Milestone
v3.1.0

Comments

@josecelano
Copy link
Member

josecelano commented May 30, 2024 •
edited
Loading

https://www.npmjs.com/package/shiki-es package has been deprecated. Migrate to https://github.com/antfu/shikiji

image
@josecelano josecelano added the Dependencies Related to Dependencies label May 30, 2024
@josecelano josecelano changed the title Migrate shikiji dep to shikiji Migrate shiki-es dep to shikiji May 30, 2024
@grmbyrn grmbyrn self-assigned this Jun 4, 2024
@grmbyrn
Copy link
Collaborator

grmbyrn commented Jun 4, 2024

Hi @josecelano I've been working on how to migrate from shiki-es to shikiji as recommended. I have found that shinjiki is also deprecated as both have been merged back into Shiki 1. However, I think that the warning is coming from Histoire and not from our implementation of it.

I checked the Histoire repo and found that a PR was made to migrate it in February but it's not passing Nuxt 3 tests so hasn't been merged.

@josecelano
Copy link
Member Author

Hi @josecelano I've been working on how to migrate from shiki-es to shikiji as recommended. I have found that shinjiki is also deprecated as both have been merged back into Shiki 1. However, I think that the warning is coming from Histoire and not from our implementation of it.

I checked the Histoire repo and found that a PR was made to migrate it in February but it's not passing Nuxt 3 tests so hasn't been merged.

OK @grmbyrn we can wait until it's merged.

@josecelano josecelano transferred this issue from torrust/torrust-website-v1 Oct 3, 2024
@josecelano josecelano added this to the v3.1.0 milestone Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@grmbyrn grmbyrn

Labels
Dependencies Related to Dependencies
Projects
None yet
Milestone
v3.1.0
Development

No branches or pull requests
2 participants
@josecelano @grmbyrn