-
Notifications
You must be signed in to change notification settings - Fork 645
Conversation
Can you add it here as well? https://github.com/Microsoft/vscode-go/blob/master/src/goInstallTools.ts#L33 |
done. can you take a look ? @ramya-rao-a |
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.
Just 2 small changes more
src/goInstallTools.ts
Outdated
@@ -25,7 +25,8 @@ function getTools(goVersion: SemVersion): { [key: string]: string } { | |||
'go-outline': 'github.com/lukehoban/go-outline', | |||
'go-symbols': 'github.com/newhook/go-symbols', | |||
'guru': 'golang.org/x/tools/cmd/guru', | |||
'gorename': 'golang.org/x/tools/cmd/gorename' | |||
'gorename': 'golang.org/x/tools/cmd/gorename', | |||
'godoc': 'golang.org/x/tools/cmd/godoc' |
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.
This should go inside the if block if (goConfig['docsTool'] === 'godoc')
, because when user choose gogetdoc
, they need not install godoc
README.md
Outdated
@@ -171,6 +172,7 @@ go get -u -v github.com/tpng/gopkgs | |||
go get -u -v github.com/newhook/go-symbols | |||
go get -u -v golang.org/x/tools/cmd/guru | |||
go get -u -v github.com/cweill/gotests/... | |||
go get golang.org/x/tools/cmd/godoc |
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.
Missing -u -v
@ramya-rao-a Fix as your request. thanks for guide me |
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.
Thanks!
Fix is out in the latest update (0.6.56) to the Go extension |
No description provided.