forked from golang/vscode-go
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
tools/goplssetting: handle changes in gopls setting structure
This CL handles the various changes in my current stack of gopls CLs and it shouldn't be merged until CL 280355 is merged. It handles the new gopls settings hierarchies, maps keyed by enums, and changes to default values. I was struggling to work with the method writing out the lines directly, so I switched to a JSON struct. This works fine in most cases, except when writing out the default values. I think gopls should probably produce interface{} for default values, but that will need to be dealt with later. For now, I added a relatively basic work-around. Change-Id: Ie4b69074f1bf02023fa39488ebc1c5c9660a03fc Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/279728 Run-TryBot: Rebecca Stambler <[email protected]> TryBot-Result: kokoro <[email protected]> Trust: Rebecca Stambler <[email protected]> Trust: Hyang-Ah Hana Kim <[email protected]> Reviewed-by: Hyang-Ah Hana Kim <[email protected]>
- Loading branch information
1 parent
343a5dc
commit de64eaa
Showing
7 changed files
with
718 additions
and
334 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.