-
Notifications
You must be signed in to change notification settings - Fork 44
Upgrade to @mdx-js/mdx v. 1 #49
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
Conversation
Thanks for this! One more request: could you remove the dependency on |
No prob. Glad I could help out with this awesome thing which was just what I needed! 😃 You're right, |
Published as 3.0.0 🎉 Thanks for your work on this! |
Glad to help out! Might also want to upgrade |
Just had an attempt at this in the mdx-macro--1 branch, although it appears that the whole transform would need to be rewritten from scratch. Given that mdx.macro doesn't really work anyway (it doesn't handle auto-reload due to a limitation in babel-plugin-macros), I'm going to skip it for now. Would be happy to accept another PR though. |
Ya using this instead of |
This will fix issues #47 and #45.
Needed to include https://www.npmjs.com/package/@mdx-js/react for things to work because of breaking changes in @mdx-js/mdx v. 1