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

Iteration Plan for May - June 2017 #959

Closed
3 of 4 tasks
ramya-rao-a opened this issue May 4, 2017 · 5 comments
Closed
3 of 4 tasks

Iteration Plan for May - June 2017 #959

ramya-rao-a opened this issue May 4, 2017 · 5 comments

Comments

@ramya-rao-a
Copy link
Contributor

ramya-rao-a commented May 4, 2017

Update:

At the end of the May milestone we released 0.6.62 which had a bunch of bug fixes and the below 2 planned items

The other 2 are now moved to June

@ramya-rao-a ramya-rao-a changed the title Draft Iteration Plan for May 2017 Iteration Plan for May 2017 May 22, 2017
@qaisjp
Copy link

qaisjp commented May 26, 2017

Glad to hear #168 has not been forgotten! Tried VS Code for the first time today and this is the only thing I've noticed missing from GoSublime so far!

@ramya-rao-a
Copy link
Contributor Author

Sorry folks, May has been a busy month for me and I couldn't get to this. Will try to get an update out early June

@ramya-rao-a ramya-rao-a changed the title Iteration Plan for May 2017 Iteration Plan for May - June 2017 Jun 9, 2017
@qaisjp
Copy link

qaisjp commented Jul 28, 2017

What's the plan for #168?

@ramya-rao-a
Copy link
Contributor Author

@qaisjp
Completions get triggered for every new word that gets typed.
We will have to parse the Go file every time the completions get triggered and find the types to offer the method constructor in the completion list.

I have a working model for it, but I am not yet happy with the "parse file for every completion model"
Because to do that I have to spawn a new process and call a Go tool.

The right way to do go about this would be using a language service (we don't have one yet that supports the auto complete feature). That way there is a service that knows the state and AST of the current file at any given time and can provide the necessary suggestion.

I'll try and get #168 in with the current working solution I have for the next update with a setting to turn it off. I am not sure what the perf implications would be when trying to spawn a new process for every word. Shouldn't be too bad, we already do that for every hover to get the hover info. Let's see.

@qaisjp
Copy link

qaisjp commented Jul 28, 2017

Okay, that sounds fair enough. I've subscribed to #168 now, so please keep us updated there if you can, and when you have the chance!

Thanks for the quick reply :)

@vscodebot vscodebot bot locked and limited conversation to collaborators Jan 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants