Skip to content

Don't collect open-file telemetry on the syntax server - #24178

Merged
Andrew Casey (amcasey) merged 1 commit into
microsoft:masterfrom
amcasey:SyntaxServerOpenFileTelemetry
May 16, 2018
Merged

Don't collect open-file telemetry on the syntax server#24178
Andrew Casey (amcasey) merged 1 commit into
microsoft:masterfrom
amcasey:SyntaxServerOpenFileTelemetry

Conversation

@amcasey

Copy link
Copy Markdown
Member

It's throwing an exception (since there's no program from which to retrieve the SourceFile) and there's no reason to fix it since the same files are open in the semantic server.

It's throwing an exception (since there's no program from which to
retrieve the SourceFile) and there's no reason to fix it since the
same files are open in the semantic server.
@amcasey

Copy link
Copy Markdown
MemberAuthor

Without this change, the syntax server is broken and a bunch of operations don't work in VS.

@amcasey
Andrew Casey (amcasey) merged commit bc79917 into microsoft:masterMay 16, 2018
@amcasey
Andrew Casey (amcasey) deleted the SyntaxServerOpenFileTelemetry branch May 16, 2018 21:10
@ghost

Copy link
Copy Markdown

👍

@microsoftMicrosoft (microsoft) locked and limited conversation to collaborators Jul 31, 2018
Sign up for freeto 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

@amcasey@mhegazy