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

Vite/MDX: Fix mdx compiler for vite #20121

Merged
merged 3 commits into from
Dec 7, 2022
Merged

Vite/MDX: Fix mdx compiler for vite #20121

merged 3 commits into from
Dec 7, 2022

Conversation

ndelangen
Copy link
Member

@ndelangen ndelangen commented Dec 7, 2022

Related: #20116

I change the vite mdx plugin to use the mdxCompilerOptions from the preset, similar to how webpack does it now.

This could definitely be improved by your PR @joshwooding
But I'd say getting the mdxCompilerOptions from the proper place in main.js and properly exposing that as a solid public API is a secondary goal.

Primary goal is to get this working in strict installation environments such a pnpm.

@ndelangen ndelangen self-assigned this Dec 7, 2022
@ndelangen ndelangen added maintenance User-facing maintenance tasks builder-vite labels Dec 7, 2022
@ndelangen ndelangen requested review from joshwooding, shilman and IanVS and removed request for joshwooding December 7, 2022 09:53
@shilman shilman changed the title Fix the mdx compiler for vite Vite/MDX: Fix mdx compiler for vite Dec 7, 2022
@shilman shilman added the mdx label Dec 7, 2022
Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

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

Great work! Go ahead & merge once CI finishes

@ndelangen ndelangen merged commit 4880bfa into next Dec 7, 2022
@ndelangen ndelangen deleted the fix/vite-docs-mdx branch December 7, 2022 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
builder-vite maintenance User-facing maintenance tasks mdx
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants