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

fix mermaid version #90

Merged
merged 4 commits into from
Jul 9, 2023
Merged

fix mermaid version #90

merged 4 commits into from
Jul 9, 2023

Conversation

paul-sachs
Copy link
Contributor

@paul-sachs paul-sachs commented May 25, 2023

Eliminate possible random errors caused by the most recent version of mermaid when it changes some output.

@sorphwer
Copy link

sorphwer commented Jul 8, 2023

@paul-sachs I appreciate your work and I verifed that this PR does fix the issue #91 . This PR should be merged ASAP but the design of pulling mermaid from CDN is quite a bad idea that has to be fixed.

For everyone who is suffering from this issue, I suggest using patch-package to made corresponding code change and add "postinstall": "patch-package" to let CICD work well.

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.

3 participants