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

Support for F# #6554

Closed
rolyv opened this issue Dec 15, 2022 · 3 comments
Closed

Support for F# #6554

rolyv opened this issue Dec 15, 2022 · 3 comments

Comments

@rolyv
Copy link

rolyv commented Dec 15, 2022

There is a small group within the F# community that is willing to work on adding this. The purpose of this issue is mainly to understand if this would be something that you would potentially consider accepting via PR to consolidate under the dotnet umbrella. And if so, would you have any guidance on what you would want to see to consider accepting such a PR?

@abelbraaksma
Copy link

Chiming in here. Currently two plugins exist (one for security, one for linting/coverage) that are a bit outdated, but could be a starting point for doing this. I've looked at it and it wasn't immediately obvious how they could be unified.

It'd be interesting to hear from the dotnet sonar team to hear what their preferred approach is to add it. I think it would go a long way to add this kind of F# support to SonarQube through this repo.

@pavel-mikula-sonarsource
Copy link
Contributor

Hi,

Please create a topic on http://community.sonarsource.com/ where it has better visibility for our product managers.

As far as I know, F# is not a Roslyn-based language and the analyzer would not live in this repository.

Our plugin SDK is also available to build custom plugin for your SonarQube instance (not SonarCloud, not SonarLint)

@abelbraaksma
Copy link

@pavel-mikula-sonarsource, you’re right, F# isn’t Roslyn based. But considering this repo is called ‘sonar-dotnet’, it feels like the right place to bring all (MS) dotnet technologies under the same roof.

But if it’s impossible to hook up a different language + analyser, that’s not possible then…

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

No branches or pull requests

3 participants