Skip to content

DO NOT MERGE Default Ruby sensor skip flag to true#116

Draft
ghislainpiot wants to merge 2 commits intomasterfrom
skip-ruby-sensor-flag
Draft

DO NOT MERGE Default Ruby sensor skip flag to true#116
ghislainpiot wants to merge 2 commits intomasterfrom
skip-ruby-sensor-flag

Conversation

@ghislainpiot
Copy link

@ghislainpiot ghislainpiot commented Mar 23, 2026

Summary

  • default the internal Ruby sensor skip flag to true
  • require sonar.ruby.internal.skipRubySensor=false to re-enable RubySensor in scanner/server analysis
  • register the property in plugin metadata and update focused unit coverage for the new default

Testing

  • ./gradlew :sonar-ruby-plugin:test --tests org.sonarsource.ruby.plugin.RubySensorTest --tests org.sonarsource.ruby.plugin.RubyPluginTest spotlessCheck

Part of

  • N/A

Add an internal property gate on RubySensor so tests can bypass the Slang-based Ruby analysis path. The change skips the sensor before parsing and adds focused unit coverage for the new fast-path.
Make the internal Ruby sensor skip flag default to true and require an explicit false value to re-enable the sensor in scanner/server analysis. Register the property in the plugin metadata and update the focused unit tests to cover the new default and the false override path.
@ghislainpiot ghislainpiot changed the title DO NOT MERGE Add internal Ruby sensor skip flag DO NOT MERGE Default Ruby sensor skip flag to true Mar 24, 2026
@sonarqube-next
Copy link

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