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 mermaid.js version to v11 #13

Merged
merged 1 commit into from
Jan 27, 2025
Merged

Conversation

mofeing
Copy link
Contributor

@mofeing mofeing commented Jan 24, 2025

There is a typo in the commit message (.js, not .jl)

@mortenpi
Copy link
Member

LGTM, thank you!

@mortenpi mortenpi merged commit 0cf4b37 into JuliaDocs:main Jan 27, 2025
11 checks passed
@mortenpi
Copy link
Member

Just one note: I think in the future we might need to think about a policy with regards to the mermaid.js major bumps. If it has changes that break how the user-provided Mermaid source is interpreted, then that would be a breaking change here too IMHO.

It does look like v11 didn't actually have any substantial braking changes to the Mermaid language though: https://github.com/mermaid-js/mermaid/releases/tag/v11.0.0

But just to be conservative, I'll tag this as 0.2.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