mrjosh / helm-ls

⎈ Language server for Helm ⎈
MIT License
220 stars 16 forks source link

feat: use helm as a libary instead of copying code #77

Closed qvalentin closed 4 months ago

qvalentin commented 5 months ago

This change would include the official helm code as a dependency instead of copying only the code used for linting etc. to this project.

I would really like to get your input @mrjosh on such a design decision. Other people are also welcome to the discussion.

Advantages:

Disadvantages:

qvalentin commented 5 months ago

This would revert https://github.com/mrjosh/helm-ls/pull/14

qvalentin commented 5 months ago

Using helm as a library does not seem uncommon, quite a lot of projects do this: https://github.com/helm/helm/network/dependents?package_id=UGFja2FnZS0yMjY3Mjg4NjYw

mrjosh commented 5 months ago

Hi @qvalentin thanks for the info regarding using helm-core as library.

By leveraging the Helm code directly, we inherently reduce the overhead associated with maintaining a separate, potentially divergent codebase. This shift not only streamlines our development process but also ensures that we benefit from the latest improvements and features added to Helm, keeping our project up-to-date and robust.

Regarding the concerns about the increased binary size and potential breaking changes, these are valid points. However, the reduction in binary size with compression to 29M is quite efficient, and by prioritizing thorough testing, we can mitigate the risks of breaking changes effectively. This proactive approach in testing will safeguard our project's stability and reliability.

qvalentin commented 5 months ago

Thanks for your opinion, I will take this as a approve for the proposed changes and will make it ready to be merged soon.