search

corenova / yang-js

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

Yang.require should account for include statement #20

Closed demonicblue closed 8 years ago

demonicblue commented 8 years ago

The require method currently only handles imports. Handling include for submodules should also work.

I'll try to fix this and create a pull request.

sekur commented 8 years ago

The include case needs a different handling because we need to parse the submodule without resolving the symbols. I'll see if I can address this issue. On Fri, Aug 5, 2016 at 2:30 AM Henrik Hugo notifications@github.com wrote:

The require method currently only handles imports. Handling include for submodules should also work.

I'll try to fix this and create a pull request.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/corenova/yang-js/issues/20, or mute the thread https://github.com/notifications/unsubscribe-auth/AA6KbJfj64Cblv0tZYSNqQeoARfGWfo_ks5qcwK0gaJpZM4Jdg3g .

Peter Lee Corenova Technologies +1 310 400 6450 peter@corenova.com