search

mbj4668 / pyang

An extensible YANG validator and converter in python
ISC License
528 stars 342 forks source link

Introduce LSP Support #904

Open esmasth opened 1 month ago

esmasth commented 1 month ago

This addresses mbj4668/pyang#903

mbj4668 commented 1 month ago

I would prefer if lsp was an "optional dependency", i.e., if I don't use lsp I should not have to install it in order to use pyang.

Also, please add some documentation (and update the README)

esmasth commented 1 month ago

Thanks @mbj4668. I can work on productization given the encouragement. Ambition level for this PR is to more or less keep to the same functionality level over LSP as much is available over CLI.

esmasth commented 4 weeks ago

@mbj4668 your comments were addressed by removing pygls installation dependency. It is still kept as a development dependency.

Documentation and tests are now added to productize the code. Request your review and trial.