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

SymbolStartAnalysisContext: Add support for VB #9028

Closed

Conversation

martin-strecker-sonarsource
Copy link
Contributor

Fixes #9077

This PR is WIP with failing tests and outcommented code. It is here to show how we could fix the issue long term.

Copy link

sonarcloud bot commented Apr 8, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@martin-strecker-sonarsource
Copy link
Contributor Author

Closed as not needed at the moment.

HINT: Please do not delete the branch. We may need it later if we want to add VB support:

https://github.com/SonarSource/sonar-dotnet/pull/9027/files#diff-4011f2fcb6ba142f62c3c762a43f3057d42d09cde05ca6f599490ce594273887R133-R135

@martin-strecker-sonarsource martin-strecker-sonarsource changed the title SymbolStartAnalysisContext: Remove dependency to SyntaxKind SymbolStartAnalysisContext: Add support for VB Apr 11, 2024
@martin-strecker-sonarsource
Copy link
Contributor Author

/azp run

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.

SymbolStartAnalysisContext: Add support for VB
1 participant