SONARRUBY-98 clean sonar issues #89

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

Merged

BartLucien merged 5 commits into master from lb/SONARRUBY-98-clean-sonar-issues

Dec 9, 2025

+367 −354

Merged

SONARRUBY-98 clean sonar issues #89

SONARRUBY-98 Transform complex use of Collect to ToList

SonarQube-Next / [Ruby] SonarQube Code Analysis succeeded Dec 5, 2025 in 1m 34s

Quality Gate passed

Issues
0 New issues
71 Fixed issues
2 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
90.4% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

Annotations

Check warning on line 157 in sonar-ruby-plugin/src/main/java/org/sonarsource/ruby/converter/RubyVisitor.java

sonarqube-next / [Ruby] SonarQube Code Analysis

Reduce the number of non-empty switch cases from 32 to at most 30.

[S1479] "switch" statements should not have too many "case" clauses
 See more on https://next.sonarqube.com/sonarqube/project/issues?id=SonarSource_sonar-ruby&pullRequest=89&issues=3a871038-18ba-4583-be8c-6a2e51484b7f&open=3a871038-18ba-4583-be8c-6a2e51484b7f

Check warning on line 253 in sonar-ruby-plugin/src/test/java/org/sonarsource/ruby/converter/RubyConverterTest.java

sonarqube-next / [Ruby] SonarQube Code Analysis

Replace these 10 tests with a single Parameterized one.

[S5976] Similar tests should be grouped in a single Parameterized test
 See more on https://next.sonarqube.com/sonarqube/project/issues?id=SonarSource_sonar-ruby&pullRequest=89&issues=62a35896-f3f1-424e-9c4e-68a011b5dda2&open=62a35896-f3f1-424e-9c4e-68a011b5dda2

View more details on SonarQube-Next