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(markdown): add support for shiki option langAlias #12039

Merged
merged 7 commits into from
Oct 9, 2024

Conversation

ematipico
Copy link
Member

@ematipico ematipico commented Sep 20, 2024

Changes

Closes withastro/roadmap#954

This PR adds support for langAlias

Testing

I added new tests

Docs

withastro/docs#9453

Copy link

changeset-bot bot commented Sep 20, 2024

🦋 Changeset detected

Latest 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

@github-actions github-actions bot added feat: markdown Related to Markdown (scope) pkg: astro Related to the core `astro` package (scope) semver: minor Change triggers a `minor` release labels Sep 20, 2024
Copy link
Contributor

@github-actions github-actions bot left a 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.

@ematipico ematipico changed the title feat(shiki): add support for langAlias feat(markdown): add support for shiki option langAlias Sep 20, 2024
Copy link
Member

@sarah11918 sarah11918 left a 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!

@bluwy bluwy added this to the 4.16 milestone Oct 1, 2024
@bluwy
Copy link
Member

bluwy commented Oct 5, 2024

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 loadLanguage and not the pre-aliased one.

@ematipico ematipico force-pushed the feat/add-shiki-lang-alias branch from a6a6995 to b26abe6 Compare October 8, 2024 11:13
Copy link
Member

@sarah11918 sarah11918 left a 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>
Copy link
Member

@sarah11918 sarah11918 left a 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!

@ematipico ematipico merged commit 710a1a1 into main Oct 9, 2024
13 checks passed
@ematipico ematipico deleted the feat/add-shiki-lang-alias branch October 9, 2024 09:51
@astrobot-houston astrobot-houston mentioned this pull request Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat: markdown Related to Markdown (scope) pkg: astro Related to the core `astro` package (scope) semver: minor Change triggers a `minor` release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants