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

Skip TestCode classification type when computing tokens. #74433

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

JoeRobich
Copy link
Member

When the AdditiveClassification TestCode is encountered the SemanticTokenHelper throws a "missing token index" exception. Which is presented as a failed range request.
image

Skipping this classification when calculating tokens allows the request to complete.
image

@JoeRobich JoeRobich enabled auto-merge July 18, 2024 14:31
@JoeRobich JoeRobich merged commit 21262e7 into main Jul 18, 2024
25 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Jul 18, 2024
@JoeRobich JoeRobich added the LSP issues related to the roslyn language server protocol implementation label Jul 19, 2024
@RikkiGibson RikkiGibson modified the milestones: Next, 17.12 P1 Jul 29, 2024
@JoeRobich JoeRobich deleted the dev/jorobich/fix-lsp-testcode branch September 24, 2024 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-IDE LSP issues related to the roslyn language server protocol implementation untriaged Issues and PRs which have not yet been triaged by a lead
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants