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

previousToken information for EarlyExitException #616

Merged
merged 1 commit into from
Dec 8, 2017

Conversation

christianvoigt
Copy link
Contributor

also added the previous token to buildEarlyExitMessage

@bd82
Copy link
Member

bd82 commented Dec 8, 2017

use:

npm run format

To format the project using prettier.
other wise it would fail the build...

@christianvoigt
Copy link
Contributor Author

I already did that.

@christianvoigt
Copy link
Contributor Author

Oh, I pushed the changes to Github, but somehow this did not work. Will do it again.

@bd82
Copy link
Member

bd82 commented Dec 8, 2017

great

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 5acf0db on christianvoigt:earlyexit-previous-token into 5315cbb on SAP:master.

@bd82 bd82 merged commit b2d81ac into Chevrotain:master Dec 8, 2017
@bd82
Copy link
Member

bd82 commented Dec 8, 2017

Thanks!

@bd82
Copy link
Member

bd82 commented Dec 8, 2017

Fixes #613

@bd82 bd82 changed the title previousToken information for EarlyExitException (https://github.com/SAP/chevrotain#613) previousToken information for EarlyExitException Dec 8, 2017
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.

None yet

3 participants