Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

invoke next handler in case if no signature help providers can be fou… #10673

Merged
merged 1 commit into from
Apr 19, 2016

Conversation

vladima
Copy link
Contributor

@vladima vladima commented Apr 19, 2016

…nd (i.e. if document for the text snapshot was not yet created). This become the issue on the TypeScript side, we analyze open files and its references in order to build the transitive closure of sources that contribute to the program. The actual Roslyn project and documents are created after the analysis is finished. As a consequence when analysis in in progress user cannot type anywhere in the file.

…nd (i.e. if document for the text snapshot cannot be found)
@vladima
Copy link
Contributor Author

vladima commented Apr 19, 2016

// cc @CyrusNajmabadi, @jasonmalinowski

@jasonmalinowski
Copy link
Member

👍. Do you want this targeting master, so it'll ship to customers sooner?

@jasonmalinowski
Copy link
Member

Actually, to simplify code flow, let's just merge this too.

@jasonmalinowski jasonmalinowski merged commit 694d5d7 into dotnet:future Apr 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants