Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Upgrade to MDX 2 #36888

Closed
remcohaszing opened this issue May 13, 2022 · 0 comments
Closed

Upgrade to MDX 2 #36888

remcohaszing opened this issue May 13, 2022 · 0 comments

Comments

@remcohaszing
Copy link
Contributor

Describe the feature you'd like to request

Currently MDX 2 is supported, but everything in Next.js is focused on supporting MDX 1. I would like to update this. I noticed this while implementing #36815.

Describe the solution you'd like

  • The with-mdx example is updated to use MDX 2
  • The with-mdx-remote example is updated to next-mdx-remote 4
  • Preferably @next/mdx uses a dependency on @mdx-js/loader instead of a peer dependency.
  • The @next/mdx peer dependency on @mdx-js/react is removed, as it’s redundant now.

I’ll gladly do this myself. Just let me know if you’ll accept a PR for this.

Describe alternatives you've considered

The alternative is not to upgrade, but I think it’s a bad idea to keep suggesting older versions of MDX.

@vercel vercel locked and limited conversation to collaborators May 13, 2022
@balazsorban44 balazsorban44 converted this issue into discussion #36890 May 13, 2022

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant