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

fix: add shiki to special case list #366

Merged
merged 12 commits into from
Oct 12, 2023
Merged

fix: add shiki to special case list #366

merged 12 commits into from
Oct 12, 2023

Conversation

souporserious
Copy link
Contributor

@souporserious souporserious commented Oct 11, 2023

Adds shiki to the special case list so grammars and themes will be traced and loaded correctly when deploying.

Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca

@souporserious souporserious changed the title add shiki to special case list fix: add shiki to special case list Oct 11, 2023
test/integration/shiki.js Outdated Show resolved Hide resolved
test/unit/shiki/input.js Outdated Show resolved Hide resolved
@souporserious
Copy link
Contributor Author

@styfle thank you for the feedback! I updated the input/output files to reflect the real-world usage. However, for some reason I'm still having trouble with tests not passing.

@styfle
Copy link
Member

styfle commented Oct 12, 2023

@souporserious I pushed a change which should fix it

@souporserious
Copy link
Contributor Author

@styfle woot, that looks great, thank you! 🙏 I can't tell why the test is still failing, but I'm wondering if we need to account for themes as well since I believe shiki will look for the theme in the example use since it's specifying nord.

Copy link
Member

@styfle styfle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I pushed a couple more changes. If tests pass, we should be good, thanks! ✅

@souporserious souporserious merged commit 1fc9e01 into main Oct 12, 2023
15 checks passed
@souporserious souporserious deleted the shiki-special-case branch October 12, 2023 21:02
@github-actions
Copy link

🎉 This PR is included in version 0.24.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants