settings: break down indexing options #1010
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As per our internal conversation yesterday, this allows us to get best of both worlds where VS Code can present nice settings UI and LS isn't limited to a flat structure.
I will raise a separate PR to update all three
terraform*
options to follow the same principles as documented and further align it with https://github.com/hashicorp/vscode-terraform/blob/main/docs/settings-migration.md , i.e.instead of
we will accept
and any future (stable, non-experimental) settings can follow these principles.