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

Commit

Permalink
Remove use of -r as upstream PR isnt merged yet #2223
Browse files Browse the repository at this point in the history
  • Loading branch information
ramya-rao-a committed Jan 9, 2019
1 parent 8133ea3 commit 43a7d68
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/goDeclaration.ts
Original file line number Diff line number Diff line change
Expand Up @@ -96,9 +96,9 @@ function definitionLocation_godef(input: GoDefinitionInput, token: vscode.Cancel
return new Promise<GoDefinitionInformation>((resolve, reject) => {
// Spawn `godef` process
const args = ['-t', '-i', '-f', input.document.fileName, '-o', offset.toString()];
if (useReceivers) {
args.push('-r');
}
// if (useReceivers) {
// args.push('-r');
// }
p = cp.execFile(godefPath, args, { env, cwd }, (err, stdout, stderr) => {
try {
if (err && (<any>err).code === 'ENOENT') {
Expand Down

2 comments on commit 43a7d68

@douglarek
Copy link

@douglarek douglarek commented on 43a7d68 Jan 10, 2019

Choose a reason for hiding this comment

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

should this be 0.8.1-beta5 ?

@ramya-rao-a
Copy link
Contributor Author

Choose a reason for hiding this comment

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

You are right.
The vsix has been updated.

Thanks for reporting!

Please sign in to comment.