Skip to content

Bump mermaid dependency to v8.13.4

Dheeraj Joshi requested to merge djadmin-bump-mermaid into master

Related issues - #345452 (closed), #345035 (closed)

What does this MR do and why?

This bumps mermaid to the latest available version to apply certain improvements and bug fixes. See, https://github.com/mermaid-js/mermaid/releases

With v8.13.4, mermaid exports ESM modules, so we need to consider the default namespace.

https://webpack.js.org/guides/code-splitting/#dynamic-imports

The reason we need default is that since webpack 4, when importing a CommonJS module, the import will no longer resolve to the value of module.exports, it will instead create an artificial namespace object for the CommonJS module. For more information on the reason behind this, read webpack 4: import() and CommonJs.

Screenshots or screen recordings

image

^ sanity test

More details in #345452 (comment 743281060)

How to set up and validate locally

Nothing special.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Dheeraj Joshi

Merge request reports