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

Update pulldown-cmark to 0.11 #2381

Merged
merged 1 commit into from
May 16, 2024
Merged

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented May 16, 2024

Changelog: https://github.com/pulldown-cmark/pulldown-cmark/releases/tag/v0.11.0

Note that math is not yet supported. However, due to the new Event variant, I decided to include math script in the search index. I don't know if that is the right thing to do, but seems like something we can figure out later if we start using the math feature.

@rustbot rustbot added the S-waiting-on-review Status: waiting on a review label May 16, 2024
@ehuss ehuss added this pull request to the merge queue May 16, 2024
Merged via the queue into rust-lang:master with commit 8884008 May 16, 2024
8 checks passed
@dylanowen
Copy link
Contributor

dylanowen commented May 17, 2024

From the view of a preprocessor this isn't a SemVer compatible change and probably needs a minor version bump:

https://github.com/dylanowen/mdbook-graphviz/actions/runs/9120113600/job/25076838288?pr=132

expected `pulldown_cmark::Event<'_>`, found `Event<'_>`

Also pulldown-cmark-to-cmark doesn't seem to support 0.11 yet

@ehuss
Copy link
Contributor Author

ehuss commented May 17, 2024

😦 Indeed. Thanks for letting me know. I did not realize the pulldown_cmark types were exposed, and we updated in 0.4.37 to 0.10 and nobody mentioned anything at the time.

I'll revert it for now. We'll probably need to do some kind of public re-export.

ehuss added a commit to ehuss/mdBook that referenced this pull request May 17, 2024
…cmark"

This reverts commit 8884008,
(rust-lang#2381) reversing
changes made to 3d6caa5.

The `pulldown_cmark` types are a public API, which I did not realize.
@dylanowen
Copy link
Contributor

Thanks for the quick fix! I think we ran into issues with 0.4.37 as well but I neglected to warn anyone here 😬:

dylanowen/mdbook-graphviz#107 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: waiting on a review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants