search

TypeFox / yang-lsp

A Language Server for YANG
http://www.yang-central.org
Apache License 2.0
51 stars 13 forks source link

bad-import-yang-version is raised for legal YANG 1.0 imports #228

Closed esmasth closed 4 months ago

esmasth commented 11 months ago

https://www.rfc-editor.org/rfc/rfc7950#section-12 specifies that YANG1.0 module importing YANG1.1 module is allowed when importing without revision. Is that yang-lsp development community's interpretation as well?

For example, iana-if-type.yang (YANG 1.0) importing ietf-interfaces.yang (YANG1.1) is raised as an error, and I would not like to suppress it globally.

esmasth commented 5 months ago

@dhuebner, should we consider it as a bugfix, since it raises a false positive error and not a warning?

dhuebner commented 4 months ago

@esmasth Can we close this now? Or is there something left?

esmasth commented 4 months ago

Ok to close. I don't have more in the pipeline in this context.