Skip to content

fix sonarqube script - #496

Merged
davewichers merged 2 commits into
OWASP-Benchmark:masterfrom
darkspirit510:fix-sonarqube-script
Aug 23, 2026
Merged

fix sonarqube script#496
davewichers merged 2 commits into
OWASP-Benchmark:masterfrom
darkspirit510:fix-sonarqube-script

Conversation

@darkspirit510

Copy link
Copy Markdown
Contributor

SonarQube now requires definition of projectBaseDir (and also deprecated how you login). Both is fixed so you can run SonarQube again - and I can review #448.

@davewichers
davewichers merged commit 0db793a into OWASP-Benchmark:master Aug 23, 2026
4 checks passed
@davewichers

Copy link
Copy Markdown
Contributor

@darkspirit510 - I noticed that both the old and new output print out MANY lines like this:

14:40:06.077 WARN Too many duplication references on file src/main/java/org/owasp/benchmark/testcode/BenchmarkTest01318.java for block at line 74. Keep only the first 100 references.
14:40:06.077 WARN Too many duplication references on file src/main/java/org/owasp/benchmark/testcode/BenchmarkTest01318.java for block at line 76. Keep only the first 100 references.

Is there any way to suppress/disable those?

I tested this PR and it seems to work identically to the existing one, with regard to the score. But I assume there is a reason you wanted to change/update it. So I went ahead and merged it.

if you can figure out how to disable all these WARNings, that would be helpful.

Thx

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.

2 participants