Branch: refs/heads/master Home: https://github.com/xwiki/xwiki-commons Commit: a4422119d041985762d0b2519f4b639b157d0041 https://github.com/xwiki/xwiki-commons/commit/a4422119d041985762d0b2519f4b63... Author: claude[bot] <209825114+claude[bot]@users.noreply.github.com> Date: 2026-09-01 (Tue, 01 Sep 2026) Changed paths: M xwiki-commons-core/xwiki-commons-observation/xwiki-commons-observation-local/src/main/java/org/xwiki/observation/internal/DefaultObservationManager.java M xwiki-commons-tools/xwiki-commons-tool-test/xwiki-commons-tool-test-simple/src/test/java/org/xwiki/test/mockito/StringReaderMatcherTest.java M xwiki-commons-tools/xwiki-commons-tool-verification-resources/src/main/java/org/xwiki/tool/checkstyle/UnstableAnnotationCheck.java Log Message: ----------- [Misc] Fix various SonarQube issues (#1940) * [Misc] Fix various SonarQube issues * java:S1172 remove unused private method parameters * java:S1130 remove an unthrowable declared exception from a private method * java:S5778 hoist the fixture out of an assertThrows() lambda Co-Authored-By: Vincent Massol <[email protected]> * [Misc] Drop the java:S1130 site rejected by the Quality gate The MockitoComponentMocker#registerMockDependencies declaration line already carried a pre-existing java:S3776 (cognitive complexity 22), so rewriting that line to remove the unthrowable "throws Exception" made the gate attribute that finding to this pull request. Reducing the method's complexity is a refactor rather than a SonarQube cleanup, so the site is dropped instead. Co-Authored-By: Vincent Massol <[email protected]> --------- Co-authored-by: Vincent Massol <[email protected]> To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-commons/settings/notifications