fix(deps): update dependency codenarc to v3.5.0

[renovate]

This PR contains the following updates:

Package	Change	Age	Adoption	Passing	Confidence
codenarc (source)	3.2.0 -> 3.5.0

---

Release Notes

CodeNarc/CodeNarc (codenarc)

v3.5.0

New Rules

• New Jenkins ruleset with 7 new rules. By default, these apply only to 'Jenkinsfile' (Daniel Zänker):
  • #​771: ParameterOrReturnTypeNotSerializable rule: Every parameter and return type has to implement the Serializable interface in Jenkins
  • #​770: ObjectOverrideOnlyNonCpsMethods rule: Overridden methods of the standard library (e.g. from java.lang.Object) are often called from there and therefore must not be CPS transformed in Jenkins.
  • #​769: ForbiddenCallInCpsMethod rule: Some methods from the standard library cannot be CPS transformed and therefore must not be called from CPS transformed methods in Jenkins.
  • #​768: ExpressionInCpsMethodNotSerializable rule: Every expression/variable in a CPS transformed method in Jenkins can potentially be serialized and should therefore implement the Serializable interface.
  • #​767: CpsCallFromNonCpsMethod rule: CPS transformed methods may not be called from non CPS transformed methods in Jenkins.
  • #​766: ClosureInGString rule: Closures are CPS transformed in Jenkins and will cause an error at runtime when used in GStrings. Typically, they can be replaced by variable interpolation.
  • #​765: ClassNotSerializable rule: Classes in Jenkins pipeline libraries and Jenkinsfiles should generally implement the Serializable interface because every expression/variable used in a CPS transformed method can potentially be serialized.
• #​772: New NonSerializableFieldInSerializableClass rule: All fields of a class that implements Serializable should also implement it. (Daniel Zänker).

Updated/Enhanced Rules and Bug Fixes

• #​777: Speed up UnusedImport rule (Iulian Dragos).
• #​762: BooleanMethodReturnsNull rule: Analyze contained closure expressions within their own context, not as part of outer method context.
• #​781: DuplicateNumberLiteral rule: Fix duplicateNumberMinimumValue handling for Long and BigDecimal values.
• #​695: Refactor directory-handling in results. (Ulrich Eckhardt)

v3.4.0

Updated/Enhanced Rules and Bug Fixes

• #​751: ClassStartsWithBlankLine rule: Fix for interfaces, enums and traits with annotations on the same line. (Sven)
• #​739: NoScriptBindings rule: Fix false positive when local variable is reassigned.
• #​745: TrailingComma rule: Do not require trailing comma if last item is followed by closing bracket on the same line for Maps (that is consistent with Lists).
• #​743: SpaceAfterCommentDelimiter and SpaceBeforeCommentDelimiter rules: Fix false negatives for dollar-slashy strings.
• #​748: UnnecessarySetter rule: Skip calls to static methods in the same class.
• #​749: UnnecessaryGString rule: Escape special chars in the violation message to reflect the actual source.
• #​758: SpaceAroundOperator rule: Fix violation for the optional array operator ?[.

Command-line

• #​754: Command-line (FilesystemSourceAnalyzer): Match includes and excludes against full source file path and also against path relative to the base directory.
• #​756: Command-line (FilesystemSourceAnalyzer): Fix relative paths on Windows. (Steven Hartland)

Miscellaneous

• #​744: Documentation: Fix wording and typos. (René Scheibe)
• #​740: Documentation: In, "Naming Rules", fix asterisks not showing up when rendering Markdown.
• #​759: Consolidate analysis results and violations for report writer tests.

v3.3.0

New Rules

• #​728: New SpockMissingAssert rule (junit) - Checks for boolean conditions intended as assertions within Spock tests that are nested within a block and are thus ignored. (Jean André Gauthier)
• #​730: New SpaceAfterCommentDelimiter rule (comments) - Checks that there is whitespace after comment characters (// and /*).
• #​737: New SpaceBeforeCommentDelimiter rule (comments) - Checks that there is whitespace before comment characters (// and /*).
• #​738: New NoScriptBindings rule (concurrency) - Checks for global variables that are bound to a script; they can cause concurrency bugs. (Josh Chorlton)

Updated/Enhanced Rules and Bug Fixes

• #​729: Indentation rule: Add optional indentUnderLabel property to allow indenting code underneath labels. (Charalampos Makrylakis)
• #​725: ImplicitReturnStatement rule: Fix false positive for return within a synchronized block.
• #​733: SpaceAfterOpeningBrace rule: Fix false positive for Groovy 4 switch expressions.
• #​726: Fixed CodeNarc command-line -sourcefiles argument to use the basedir parameter (if specified), and work with paths relative to the current directory. Changed FilesSourceAnalyzer to default baseDirectory to ‘.’ rather than System.getProperty('user.dir').

Documentation

• codenarc-other-tools-frameworks: Added link to #​630 for info on controlling the version of Groovy that CodeNarc uses for the Gradle plugin.

---

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

---

• If you want to rebase/retry this PR, check this box

---

This PR was generated by Mend Renovate. View the repository job log.

[github-actions]

❌ Gradle Build failed: https://gradle.com/s/fbczepuh6yzjo

[github-actions]

❌ Gradle Build failed: https://gradle.com/s/yo7mylrcx75rq

[github-actions]

❌ Gradle Build failed: https://gradle.com/s/cge53eoslfx4y