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

Analyzer: VB support #22085

Open
roji opened this issue Aug 16, 2020 · 0 comments
Open

Analyzer: VB support #22085

roji opened this issue Aug 16, 2020 · 0 comments

Comments

@roji
Copy link
Member

roji commented Aug 16, 2020

Our analyzer currently depends on C# and VB language nugets, but one of these languages isn't in use. See #21996 (comment) for a possible way to do this (/cc @jmarolf)

The VB capabilities of the analyzer are being removed for 5.0 via #22211, because of issues with referencing both language assemblies (see #22177). @sharwell has submitted #22195 (thanks!), which splits the analyzer and allows proper support for both C# and VB. Aside from that PR, for better VB support we'd identify more VB syntax patterns and narrow them down when reporting diagnostics (as we already do with C#).

@ajcvickers ajcvickers added this to the Backlog milestone Aug 17, 2020
@roji roji changed the title Analyzer: reference VB/C# Roslyn assemblies only when needed Analyzer: VB support Aug 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants