[xwiki/xwiki-dev-llm] 2e65b2: [Misc] Document five more SonarQube rules and thre...
Branch: refs/heads/claude/cool-tesla-f9ddtn Home: https://github.com/xwiki/xwiki-dev-llm Commit: 2e65b28c7ebcd6b980c013790bb4463c61e97e62 https://github.com/xwiki/xwiki-dev-llm/commit/2e65b28c7ebcd6b980c013790bb446... Author: Vincent Massol <vincent@massol.net> Date: 2026-08-01 (Sat, 01 Aug 2026) Changed paths: M xwiki/okf/sonarqube/index.md M xwiki/okf/sonarqube/modernization-rules.md M xwiki/okf/sonarqube/simplification-rules.md M xwiki/okf/sonarqube/syntax-rules.md M xwiki/okf/sonarqube/test-code-rules.md Log Message: ----------- [Misc] Document five more SonarQube rules and three instanceof scripting traps * Add S6068 (useless Mockito eq()) to the test-code family, including why the whole statement must be unwrapped and when eq() is genuinely required. * Add S2209 to the syntax family and explain why it is safe where the denylisted S3252 is not. * Add S1264, S3012 and S3024 to the simplification family. * Record the three traps that bite a scripted S6201 batch: the parenthesized cast only owns its outer paren before a '.', brace counting must start at the '{' of a "} else if" line, and \b does not match after an array type's ']'. * Record the two extra S6126 drop conditions (\r\n, and a fixture with no content line at the baseline indent) and that a whole module's pool can be one drop. * Deny S6035 on public constants (Revapi constantValueChanged) and S3824 where the guarded block does more than the put. Co-Authored-By: Vincent Massol <vincent@massol.net> To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-dev-llm/settings/notifications
participants (1)
-
XWiki Notifications