Michael Noah

Added a PR for the limiting indexing to current package. I think it could coexist with an additional setting for directories to exclude entirely from the walk as @greenstatic is...

@firelizzard18 Is it expected to be lazy in loading the list of packages, or just the tests within those packages? It looks like the current behavior is as follows, and...

I think we could address the separation of concerns issue by moving the conditional logic into the resolver, creating a `GoTestResolver.processCreatedFile` method which conditionally calls `getFile` if the file is...

@firelizzard18 @suzmue @hyangah - Just wanted to see if there are any thoughts on the above?

@firelizzard18 thanks for reaching out, I'll take a look.

Sorry for the delay, I've taken a look at this and have some ideas. Since I see you already have a spot to set the discovery mode, a couple of...

> From the perspective of those commands, any packages that are in the same module as def **and already returned in a past [driverResponse](https://github.com/golang/tools/blob/e9bb7d36c0606a2c422dffe30db7d628ebeb9302/go/packages/packages.go#L177)** (**including abc, foo, bar but not...

> Yes, that is certainly possible. I assume you are using the release versions from Market Place and not a more recent version from master? If so, I will make...

@hyangah @jamalc - Just wanted to see if you have any feedback on this issue? We have a working version on our fork and wanted to confirm that you would...

We ended up using a fork. This is a bit out of date (https://github.com/uber/vscode-go) but we keep it up to date internally - basically using a modified version of the...