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

Feature/netsupport #10

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

tomek-anuszkiewicz
Copy link

Please read commits descriptions.

Tomek Anuszkiewicz added 4 commits February 22, 2021 15:57
Only minimal set of plugins, only to version when we doesn't get any error.
In example:
In metrics file's we have:
"file": "src/vcsparser.core/DailyCodeChurnProcessor.cs"
In `Sonar scanner CLI` we can find in `context.fileSystem()` file with exact `inputFile.filename()`.
In `Sonar scanner for Net` we find entry with:
inputFile.filename() = DailyCodeChurnProcessor.cs 
inputFile.uri() = file:///C:/Repositories/SQ/vcsparser/src/vcsparser.core/DailyCodeChurnProcessor.cs 
So in case of `Net` we need to match file name from metric to uri.
We are doing this always when straightforward matching fail.
So it runs only once. In sonar we have project, and project can have modules. If `Sensor` is not global it will run on every module. Important for `Sonar Scanner for Net` where every `csproj` is a module. Because we are applying metrics from one file, taken for one repository, it make sense to mark this plugin global.
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.

1 participant