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

Upgrade Jison Repo #1011

Closed
wants to merge 1 commit into from
Closed

Conversation

klemmchr
Copy link
Member

Resolves #873

@@ -16,7 +16,7 @@
[\n]+ return 'NL';
\s+ /* skip whitespace */
\#[^\n]* /* skip comments */
\%%[^\n]* /* skip comments */
\%"%"[^\n]* /* skip comments */
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Odd. Why did this need to be enclosed in " here, and not in other diagrams?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After reviewing further, it looks like only classDiagram didnt need this?

@jgreywolf
Copy link
Contributor

Closing this PR due to the age, and how out of date with current codebase it now is

@jgreywolf jgreywolf closed this Feb 25, 2021
@knsv knsv deleted the feature/873_upgrade_jison_repo branch July 5, 2021 08:42
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.

Upgrade to newer jison repo
2 participants