Skip to content

Comments

fix: elk mermaid bundle#733

Merged
jsladerman merged 2 commits intomainfrom
jake/fix-elk
Nov 7, 2025
Merged

fix: elk mermaid bundle#733
jsladerman merged 2 commits intomainfrom
jake/fix-elk

Conversation

@jsladerman
Copy link
Contributor

follow up on #732 , uses @mermaid-js/layout-elk directly instead of trying to fork it which didn't seem to work, and just forces elk to resolve to 0.11.0 to avoid the duplication issue. also cleans up some of the loading/initialization logic

@jsladerman jsladerman requested a review from a team as a code owner November 7, 2025 05:24
@jsladerman jsladerman added the bug Something isn't working label Nov 7, 2025
@github-actions
Copy link

github-actions bot commented Nov 7, 2025

Visit the preview URL for this PR (updated for commit 612e3f4):

https://pluralsh-design--pr733-jake-fix-elk-dj1kimb8.web.app

(expires Fri, 14 Nov 2025 06:38:05 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 784914c934330f8d0a9fd65c68898b3988262b7d

@socket-security
Copy link

socket-security bot commented Nov 7, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Added@​mermaid-js/​layout-elk@​0.2.0971009494100
Updatedmermaid@​11.4.1 ⏵ 11.12.198 +1100 +13100 +196100

View full report

@jsladerman jsladerman merged commit 9b4a234 into main Nov 7, 2025
13 checks passed
@jsladerman jsladerman deleted the jake/fix-elk branch November 7, 2025 15:13
@plural-bot
Copy link
Contributor

🎉 This PR is included in version 5.31.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants