-
-
Notifications
You must be signed in to change notification settings - Fork 7.1k
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
Requiring from CommonJS is broken #2559
Comments
This might have been fixed in 8.13.3 |
I'm facing this issue in 8.14.0
|
@kilgarenone can you share a repro? |
Hi @sidharthv96 here you go https://github.com/kilgarenone/mermaidcjs 🙏 |
Can you check this project and use the build settings we use there? |
I believe that works because it imports the esm version of mermaid. Issue at hand arises when we EDIT: I believe there's nothing can be done on mermaid side coz d3 only output ESM of their lib...unlike mermaid - both |
@kilgarenone sadly your link can no longer be found. Does the repo still exist? |
The simple answer is that we will not be supporting "require" from cjs |
Describe the bug
To Reproduce
Expected behavior
Code Sample
When you install this from a Node app and
require('mermaid')
, it fails with this error:Desktop (please complete the following information):
Smartphone (please complete the following information):
Additional context
n/a
The text was updated successfully, but these errors were encountered: