Branch: refs/heads/claude/nifty-bell-09kzzr Home: https://github.com/xwiki/xwiki-commons Commit: 268cd294466fa802a526c23009f8b38468ff37e3 https://github.com/xwiki/xwiki-commons/commit/268cd294466fa802a526c23009f8b3... Author: Vincent Massol <[email protected]> Date: 2026-09-10 (Thu, 10 Sep 2026) Changed paths: M xwiki-commons-core/xwiki-commons-velocity/src/main/java/org/xwiki/velocity/internal/util/VelocityParser.java Log Message: ----------- [Misc] Fix the SonarQube java:S2386 issues of VelocityParser * The rule fires on the mutable value of a public static final field, not on its declaration, so making the five VELOCITYDIRECTIVE_* sets immutable clears it without the visibility reduction the issue message suggests, which would be a Revapi break. Co-Authored-By: Vincent Massol <[email protected]> Co-Authored-By: Claude Opus 5 <[email protected]> Claude-Session: https://claude.ai/code/session_01ChPn88cFpNHMHk8Y2i1XB2 Commit: f8ad734dba1696b303ba9839002313bd6f1ef357 https://github.com/xwiki/xwiki-commons/commit/f8ad734dba1696b303ba9839002313... Author: Vincent Massol <[email protected]> Date: 2026-09-10 (Thu, 10 Sep 2026) Changed paths: M xwiki-commons-core/xwiki-commons-velocity/src/main/java/org/xwiki/velocity/internal/util/VelocityParser.java Log Message: ----------- [Misc] Use SetUtils#union instead of a hand-rolled union helper * commons-collections4 is already a declared dependency of the module. Co-Authored-By: Vincent Massol <[email protected]> Co-Authored-By: Claude Opus 5 <[email protected]> Claude-Session: https://claude.ai/code/session_01ChPn88cFpNHMHk8Y2i1XB2 Compare: https://github.com/xwiki/xwiki-commons/compare/268cd294466f%5E...f8ad734dba1... To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-commons/settings/notifications