-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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(markdown): add support for shiki
option langAlias
#12039
Conversation
🦋 Changeset detectedLatest commit: 421ec4c The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is blocked because it contains a minor
changeset. A reviewer will merge this at the next release if approved.
langAlias
shiki
option langAlias
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks cool, @ematipico ! I checked out the original feature documentation, and made a suggestion below for your consideration!
Actually it's our own bug 😅 I went ahead and push a commit to fix it. The idea is to use the aliased lang for |
Co-authored-by: Bjorn Lu <bjornlu.dev@gmail.com>
a6a6995
to
b26abe6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! I made a suggestion to the opening paragraph so that the link is attached to the act of creating an alias, not our config option. Also threw in that use case we talked about to give us a good excuse to link to Shiki's built-in languages!
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving for docs! Ready for 4.16!
Changes
Closes withastro/roadmap#954
This PR adds support for
langAlias
Testing
I added new tests
Docs
withastro/docs#9453