-
Notifications
You must be signed in to change notification settings - Fork 180
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add experimental Prefill Required Fields feature #799
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you confirm that this would not require us to delay releasing this option as part of next extension version? LS will actually raise warnings when unknown config options are passed.
Unless it already works that way I think we could avoid sending the option there entirely if it's (default) false
, so that it only affects people who explicitly opt-in and do so too early, before upgrading LS.
Adding to the existing With that in mind, I think this is safe to release at the same time as the LS |
This adds a new experimental setting for prefilling required attributes as blocks are completed from intellisense. For example, completing a resource provider will fill in the required attributes automatically.
091b633
to
c0369f1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Needs
hashicorp/hcl-lang#89
hashicorp/terraform-ls#657
Example