Skip to content

Conversation

aigjermo
Copy link

@aigjermo aigjermo commented Nov 2, 2018

Solves #34.

This is just the bare minimum to get it working, I haven't added any config options to opt out or anything like that.

@eastwood
Copy link

This is a decent PR. I actually just came here to implement this and was happy to see that someone beat me to it. previm supports mermaid by default and without config options as well. This would be nice to see in master

@titanism
Copy link

titanism commented Nov 3, 2023

Can we please have this merged and released to npm @shime ?

@titanism
Copy link

titanism commented Nov 3, 2023

Actually this PR is broken. It does not work.

The reason why is because 0e8b97f#diff-2f062de8635f61fbec7693ba8f7bdc5dd196f7e694240826c4b304a1e83baff3 is referencing a file path. You cannot do this. The actual file must be included, similar to Zepto.

@shime if you give me PR/GitHub/npm access we will fix this properly.

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