-
Notifications
You must be signed in to change notification settings - Fork 9
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: migrate export #1009
feat: migrate export #1009
Conversation
f505402
to
ded4ab0
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.
One question, but otherwise this looks good to me!
import mdastV6 from './mdastV6'; | ||
|
||
const migrate = (doc: string): string => { | ||
return mdx(mdastV6(doc)).replaceAll(/ /g, ' '); |
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.
So is this using the newly integrated compat processor behind the scenes?
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.
Yup yup yup. mdastV6
is the same thing, and this migrate
export is just wrapping it all into one.
This PR was released!🚀 Changes included in v7.9.0 |
🧰 Changes
Adds a
migrate
export and moves areplaceAll
into this repo.🧬 QA & Testing