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

Make a special CodeAnalysis diagnostics refresh request for Run Code Analysis command #70292

Closed
wants to merge 6 commits into from

Conversation

mavasani
Copy link
Contributor

@mavasani mavasani commented Oct 9, 2023

Strongly recommend to review commit-by-commit

Follow-up to #70186

Builds on top of LSP client PR that adds a new code analysis diagnostic refresh request

Demo

Run code analysis experience with VSIXes installed from this PR and LSP client PR

RunCodeAnalysis_LSP_CodeAnalysisRefresh

@mavasani mavasani added Area-IDE LSP issues related to the roslyn language server protocol implementation labels Oct 9, 2023
@mavasani mavasani requested a review from a team as a code owner October 9, 2023 12:01
@dotnet-issue-labeler dotnet-issue-labeler bot added the untriaged Issues and PRs which have not yet been triaged by a lead label Oct 9, 2023
@mavasani
Copy link
Contributor Author

mavasani commented Nov 6, 2023

Closing this @kayle has a separate approach that tackles this issue.

@mavasani mavasani closed this Nov 6, 2023
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.

1 participant