LesterLyu / fast-formula-parser

Parse and evaluate MS Excel formula in javascript.
https://www.npmjs.com/package/fast-formula-parser
MIT License
462 stars 64 forks source link

Capture lexer errors #17

Closed rmdort closed 4 years ago

rmdort commented 4 years ago

Shall i also throw Chevrotain Error in standard FormulaError format?

throw this.utils.formatChevrotainError(error)

LesterLyu commented 4 years ago

I'll take a look tomorrow. Lexing error are also included: https://github.com/LesterLyu/fast-formula-parser/blob/master/grammar/lexing.js#L280

LesterLyu commented 4 years ago

@rmdort Please have a look of the new release: https://github.com/LesterLyu/fast-formula-parser/releases/tag/1.0.13

rmdort commented 4 years ago

Cool thanks. That works.