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(build): don't hard fail on unknown languages in fences #1750

Merged
merged 4 commits into from
Jan 14, 2023

Conversation

clark-cui
Copy link
Contributor

bug

When a language that isn't registered by Shiki is used, page will be crashed and redict to 404.
截屏2023-01-02 16 54 36

fix

use defaultLang when language isn't registered, and give a warning log.

@clark-cui clark-cui closed this Jan 2, 2023
@clark-cui clark-cui reopened this Jan 2, 2023
@clark-cui
Copy link
Contributor Author

Is there any update? Do I need to provide more information about this?

@brc-dd
Copy link
Member

brc-dd commented Jan 13, 2023

No, sorry, I was busy last few days. Will get this merged in 1-2 days. Thanks for the PR!

@brc-dd brc-dd self-assigned this Jan 13, 2023
@brc-dd brc-dd changed the title fix: language label caused 404 feat(build): don't hard fail on unknown languages in fences Jan 14, 2023
@brc-dd brc-dd merged commit 1ae0596 into vuejs:main Jan 14, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants