You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
First, thanks for making this package, it's really helpful!
ElDoc integration would be really nice for reading existing Terraform code, e.g. having resource and argument documentation show up in the echo area for where the point is at.
I'm thinking that having the all the documentation already parsed in company-terraform-data.el would help in implementing this. Or maybe ElDoc support for Terraform belongs in a different package?
The text was updated successfully, but these errors were encountered:
I am not sure if this package is he right place to implement eldoc integration, it feels out-of-scope for a completion plugin. I do agree, however, that the resource data (as well as context detection and similar utilities) might be useful for many packages, not just company-terraform and for that reason it might deserve to be separated into a shared package. As far as I can tell, development on terraform-mode is pretty much stopped, so maybe I need to consider creating a separate "terraform-data" provider, on which both company-terraform and documentation packages might depend on.
Or I could just evolve this package into a multi-featured terraform IDE.
So there are multiple ways to go forward. I'll thing about these options.
In the meantime I will investigate how to integrate with eldoc. You can also use company-quickhelp to display the documentation on completion.
First, thanks for making this package, it's really helpful!
ElDoc integration would be really nice for reading existing Terraform code, e.g. having resource and argument documentation show up in the echo area for where the point is at.
I'm thinking that having the all the documentation already parsed in
company-terraform-data.el
would help in implementing this. Or maybe ElDoc support for Terraform belongs in a different package?The text was updated successfully, but these errors were encountered: