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

Investigate if we can enable the automatic coverage import for the .NET build as we do for .NET Framework #1813

Open
costin-zaharia-sonarsource opened this issue Dec 4, 2023 · 0 comments
Labels

Comments

@costin-zaharia-sonarsource
Copy link
Member

costin-zaharia-sonarsource commented Dec 4, 2023

Description

Automatic coverage detection and conversion runs only for the .NET Framework package. See:

#if NETFRAMEWORK
ProcessCoverageReport(config, Path.Combine(config.SonarConfigDir, FileConstants.ConfigFileName), propertyResult.FullPropertiesFilePath);
#endif

Technically we can enable this for the .NET package too. We need to understand what's the impact of enabling this and decide if we want to change how it works.

The .NET Framework build is executed when running the AzureDevOps extension tasks on Windows machines. The .NET build is executed on the other operating systems (Ubuntu, macOS, ...)

Another question is if we really need both of them. Can't we include in the extension only the dotnet package and run it everywhere?

We should also improve the documentation, as requested by the community - for example here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants