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.

Sign up for GitHub

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

Jump to bottom

Remove duplicated properties from sonar-project.properties #1019

Closed
pavel-mikula-sonarsource opened this issue May 18, 2021 · 0 comments · Fixed by #1047
Closed

Remove duplicated properties from sonar-project.properties #1019

pavel-mikula-sonarsource opened this issue May 18, 2021 · 0 comments · Fixed by #1047
Assignees
@pavel-mikula-sonarsource
Labels
Type: Bug
Projects
S4NET Kanban
Milestone
5.3.0

Comments

@pavel-mikula-sonarsource
Copy link
Contributor

sonar-project.properties files contains this section for modules. There's property with only the first record followed by the same property with full multiline value.

06ECAB71-BB0F-69DE-ADFD-C8FEA23C46E6.sonar.cs.analyzer.projectOutPaths=/home/vsts/work/1/.sonarqube/out/20
06ECAB71-BB0F-69DE-ADFD-C8FEA23C46E6.sonar.cs.roslyn.reportFilePaths=/home/vsts/work/1/s/src/Something/bin/Release/net5.0/Something.dll.RoslynCA.json
06ECAB71-BB0F-69DE-ADFD-C8FEA23C46E6.sonar.cs.analyzer.projectOutPaths=\
"/home/vsts/work/1/.sonarqube/out/20",\
"/home/vsts/work/1/.sonarqube/out/27",\
"/home/vsts/work/1/.sonarqube/out/14"
06ECAB71-BB0F-69DE-ADFD-C8FEA23C46E6.sonar.cs.roslyn.reportFilePaths=\
"/home/vsts/work/1/s/src/Something/bin/Release/net5.0/Something.dll.RoslynCA.json"

Is there a reason for this? It looks like a bug to me, as old projectOutPath (no s) was removed and projectOutPaths should be understood everywhere as a multiline value.

Same applies to sonar.cs.analyzer.projectOutPaths as well as sonar.cs.roslyn.reportFilePaths
@pavel-mikula-sonarsource pavel-mikula-sonarsource added this to To do in S4NET Kanban via automation Aug 9, 2021
@pavel-mikula-sonarsource pavel-mikula-sonarsource added this to the 5.3.0 milestone Aug 9, 2021
@pavel-mikula-sonarsource pavel-mikula-sonarsource moved this from To do to In progress in S4NET Kanban Aug 12, 2021
@pavel-mikula-sonarsource pavel-mikula-sonarsource changed the title Duplicate properties in sonar-project.properties` Remove duplicated properties in sonar-project.properties` Aug 12, 2021
@pavel-mikula-sonarsource pavel-mikula-sonarsource changed the title Remove duplicated properties in sonar-project.properties` Remove duplicated properties from sonar-project.properties Aug 12, 2021
Merged
@pavel-mikula-sonarsource pavel-mikula-sonarsource moved this from In progress to Review in progress in S4NET Kanban Aug 12, 2021
@csaba-sagi-sonarsource csaba-sagi-sonarsource linked a pull request Aug 13, 2021 that will close this issue
Remove duplicated properties from sonar-project.properties #1047
Merged
@csaba-sagi-sonarsource csaba-sagi-sonarsource moved this from Review in progress to In progress in S4NET Kanban Aug 13, 2021
@pavel-mikula-sonarsource pavel-mikula-sonarsource moved this from In progress to Review in progress in S4NET Kanban Aug 13, 2021
@csaba-sagi-sonarsource csaba-sagi-sonarsource moved this from Review in progress to Reviewer approved in S4NET Kanban Aug 13, 2021
@pavel-mikula-sonarsource pavel-mikula-sonarsource moved this from Reviewer approved to Done in S4NET Kanban Aug 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pavel-mikula-sonarsource pavel-mikula-sonarsource

Labels
Type: Bug
Projects
S4NET Kanban
  
Done
Milestone
5.3.0
Development

Successfully merging a pull request may close this issue.

Remove duplicated properties from sonar-project.properties SonarSource/sonar-scanner-msbuild
2 participants
@pavel-mikula-sonarsource @csaba-sagi-sonarsource