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

Upgrade/mdast util math #173

Merged
merged 2 commits into from
Dec 7, 2023
Merged

Conversation

squid-ney
Copy link
Contributor

@squid-ney squid-ney commented Dec 6, 2023

Hi there!

I'm working on a project that uses remark-gfm and other remark libraries as well as this library 😄. Recently, when upgrading remark-gfm to version "^4.0.0" there was an issue which was traced to different versions of the transitive dependency,mdast-util-to-markdown, between remark-gfm@^4.0.0 and [email protected]. In this PR, I am bumping the mdast-util-math version to v3, which uses v2 of mdast-util-to-markdown.

This is my first time contributing to this library. I ran tests locally and manually tested using the storybook playground- didn't see any breaking changes with this upgrade. Let me know if there is more I should do!

@squid-ney squid-ney marked this pull request as ready for review December 6, 2023 21:35
Copy link
Owner

@inokawa inokawa left a comment

Choose a reason for hiding this comment

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

Thanks!

@inokawa inokawa merged commit ed59a31 into inokawa:master Dec 7, 2023
1 check failed
This was referenced Dec 7, 2023
@squid-ney squid-ney deleted the upgrade/mdast-util-math branch December 7, 2023 13:43
@squid-ney
Copy link
Contributor Author

Hey there, saw that you merged this PR and that you opened a new related PR to support mdast v4: #174. I'm curious why the v4 support PR was closed. Also, what is the release timeline for these changes?

@inokawa
Copy link
Owner

inokawa commented Dec 7, 2023

I've noticed your fix was not enough because there are many breaking changes in mdast v5(@types/mdast : 4).
It will be released in a few days.

@inokawa
Copy link
Owner

inokawa commented Dec 7, 2023

Released in 0.9.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants