Skip to content
This repository has been archived by the owner on Jul 15, 2023. It is now read-only.

Fix godoc command not found #854

Merged
merged 3 commits into from Mar 14, 2017
Merged

Fix godoc command not found #854

merged 3 commits into from Mar 14, 2017

Conversation

ghost
Copy link

@ghost ghost commented Mar 10, 2017

No description provided.

@ramya-rao-a
Copy link
Contributor

@ghost
Copy link
Author

ghost commented Mar 13, 2017

done. can you take a look ? @ramya-rao-a

Copy link
Contributor

@ramya-rao-a ramya-rao-a left a 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

@@ -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'
Copy link
Contributor

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
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing -u -v

@ghost
Copy link
Author

ghost commented Mar 13, 2017

@ramya-rao-a Fix as your request. thanks for guide me

Copy link
Contributor

@ramya-rao-a ramya-rao-a left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ramya-rao-a ramya-rao-a merged commit dbaceea into microsoft:master Mar 14, 2017
@ramya-rao-a
Copy link
Contributor

Fix is out in the latest update (0.6.56) to the Go extension

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants