-
-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
Upgrade to newer jison repo #873
Comments
This comment has been minimized.
This comment has been minimized.
This might not be quite as trivial as I had assumed; just installing the new package, |
Just wanted to leave a quick note that I haven't made any progress on this, and probably won't, in case anyone else is thinking about it. |
I just had a look into that. This will contain a lot of breaking changes that could be problematic with features developed in other branches. I managed to resolve the first issues but the state diagram is creating this strange error
Any idea what this could be @knsv? I have a rough idea but I had no deeper look into the state diagram language definition yet. |
The corresponding PR has been closed due to the extreme age of the changes. |
The current version of jison used by mermaid is not maintained. There is a fork with some 1000 more commits under its belt. Letys switch to that version instead.
https://github.com/GerHobbelt/jison
The text was updated successfully, but these errors were encountered: