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.

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(engine-js): update deps, support new languages #875

Merged
merged 1 commit into from
Dec 27, 2024

Conversation

slevithan
Copy link
Collaborator

@slevithan slevithan commented Dec 27, 2024

Bumps oniguruma-to-es to 0.9.0, moving C++ and Apex from Mismatched to Supported in the JS engine report.

These improvements to oniguruma-to-es required complex changes in the base library and a couple dependencies. Heads up that this version also externalized dependencies in oniguruma-to-es's ESM bundle.

There are only two languages left under Mismatched: Kusto and NGINX.

  • Kusto is blocked by an Oniguruma bug (see details). The JS engine is actually the one producing the correct results.
  • Running the NGINX sample through compare.test.ts produces identical output for the JS and WASM engines (even though the playground does show small differences in highlighting). I'm not sure how to investigate further.

Copy link

netlify bot commented Dec 27, 2024

Deploy Preview for shiki-matsu ready!

Name Link
🔨 Latest commit 66bb9df
🔍 Latest deploy log https://app.netlify.com/sites/shiki-matsu/deploys/676e21825854400008bac679
😎 Deploy Preview https://deploy-preview-875--shiki-matsu.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Dec 27, 2024

Deploy Preview for shiki-next ready!

Name Link
🔨 Latest commit 66bb9df
🔍 Latest deploy log https://app.netlify.com/sites/shiki-next/deploys/676e21823331be0008667200
😎 Deploy Preview https://deploy-preview-875--shiki-next.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@antfu antfu merged commit c3524b2 into shikijs:main Dec 27, 2024
12 checks passed
@slevithan slevithan deleted the v090 branch December 27, 2024 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants