search

corenova / yang-js

YANG parser and composer
Apache License 2.0
56 stars 18 forks source link

Parser should detect syntactic errors #4

Closed pnuzhdin closed 8 years ago

pnuzhdin commented 8 years ago

I know that you are using https://www.npmjs.com/package/yang-parser which doesn't provide syntactic or semantic checks but it's too hard to use a tool like yang-js without that feature.

sekur commented 8 years ago

Hi Pavel, yes agreed, I'm currently preparing an update with 0.14.0 release to collapse the parse/preprocess/compile operation to be a singular routine so that the necessary checks take place every time. I initially thought it may be useful to expose the underlying parse function from yang-parser more or less as-is but I don't think it serves much utility that way.

pnuzhdin commented 8 years ago

@saintkepha good to know. Btw, nice project!

sekur commented 8 years ago

Thanks for the great feedback. I think I'm just a few days away from the major 0.14 release. I'll let you know once it's released - it would be very good to get additional feedback from you.

sekur commented 8 years ago

0.14.0 published, please give it a try