-
Notifications
You must be signed in to change notification settings - Fork 248
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
Cli error rendering stateDiagram on MacOs #503
Comments
I just had something like this... You have |
@tylerlong could the old repo be made more explicit about avoiding this confusion? warning signs, npm package deletion or modification to show a warning right away if installed? |
@styts you're totally right, I was stuck against old repo, didn't realize there are two different repos. Removing the old and installing new one solved my issue. Many thanks for pointing me there. |
https://www.npmjs.com/package/mermaid.cli (v0.5.1) : installation cli still mention the old repo, how are people supposed to know ? Is there someone updating those packages and wiki pages ? |
@Kochise : the readme has been updated in the mermaid.cli repo https://github.com/mermaidjs/mermaid.cli/ |
@MindaugasLaganeckas There is still a confusion. I also made the mistake of installing from the old repo. I saw the updated readme and clicked on the link, but the new repo readme starts with mmdc commands, and I even scrolled a bit but did not see the install commands. I jumped back, expecting that I needed the install commands from the first repo. Then, only npm worked for me, but at least, it worked in parts (except for the autonumber). The same outdated command is on the official npm site which is as outdated as the outdated old repo which made it even clearer for me that the command had to be right. The install commands also did not seem strange or outdated as regards the naming. When I jumped between the repos, I did not look at the version. I have lost an hour or two with this. Stupidity, but it happens, especially with 50 tabs open. It would be best if there was a comment directly on top of the old install commands of yarn and npm saying that these commands are outdated. |
Transferring issue to cli repo |
Duplicate of #433 |
Describe the bug
Error generating image from .mmd with mmdc installed via npm when input file starts with a state diagram:
To Reproduce
Steps to reproduce the behavior:
Test.mmd file content:
Command line:
Expected behavior
Test.png is rendered
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: