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

Fix godoc command not found#854

Merged
ramya-rao-a merged 3 commits intomasterfrom
unknown repository
Mar 14, 2017
Merged

Fix godoc command not found#854
ramya-rao-a merged 3 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Mar 10, 2017

No description provided.

@ramya-rao-a
Copy link
Copy Markdown
Contributor

@ghost
Copy link
Copy Markdown
Author

ghost commented Mar 13, 2017

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

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

'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
Copy Markdown
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
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
Copy Markdown
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
Copy Markdown
Author

ghost commented Mar 13, 2017

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

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