Bump dependency net.sourceforge.pmd:pmd-javascript to v7.25.0#1881
Conversation
☀️ Quality MonitorTests Coverage for New Code 〰️ Line Coverage: 100.00% — perfect 🎉 Coverage for Whole Project 〰️ Line Coverage: 92.03% — 64 missed lines Style Bugs API Problems🚫 Revapi: No warnings Vulnerabilities🛡️ OWASP Dependency Check: No vulnerabilities Software Metrics 🌀 Cyclomatic Complexity: 397 (total) 🚦 Quality GatesOverall Status: ✅ SUCCESS✅ Passed Gates
Created by Quality Monitor v4.14.0 (#2a0fad1). More details are shown in the GitHub Checks Result. |
1 participant
This PR contains the following updates:
7.24.0→7.25.0Release Notes
pmd/pmd (net.sourceforge.pmd:pmd-javascript)
v7.25.0: PMD 7.25.0 (29-May-2026)29-May-2026 - 7.25.0
The PMD team is pleased to announce PMD 7.25.0.
This is a minor release.
Table Of Contents
🚀️ New and noteworthy
Updated ANTLR library to 4.13.2
We have updated the ANTLR library (parser generator) from 4.9.3 to the latest version 4.13.2,
in order to be able to use the latest version of Apex parser library.
This is an incompatible update: In case you use custom language modules based on ANTLR, you
need to make sure to regenerate all of your lexers and parsers with the new ANTLR version.
For the ANTLR based language modules, that PMD ships (kotlin and swift and various CPD modules),
this is already done.
🌟️ New and Changed Rules
New Rules
JUnitJupiterTestNoPrivateModifierfind JUnit test classes andmethods that are private. Test classes, test methods, and lifecycle methods are not required to be public,
but they must not be private. Otherwise, they won’t be found by the test framework.
UnnecessaryBlockreports blocks that are unnecessary asthey don't introduce a new scope. This rule helps simplify code structure by identifying and flagging
redundant blocks that can make code harder to read and may be misleading.
VariableDeclarationUsageDistanceflags local variables that are declaredfar from their usage, which can make code harder to read. The rule has a property
maxDistancethat allows toconfigure the maximum allowed distance between declaration and usage.
AssertStatementInTestdetects usages ofassertstatement in tests.These should be replaced by framework assertion methods such as
assertEquals.Such methods provide better error messages and make test behave correctly when running without
-ea.Changed Rules
OnlyOneReturnhas a new propertyallowGuardIfs. When this property istrue, then guard ifs at the beginning of a method are allowed their return statements don't count.
UseUtilityClassandClassNamingConventionsnow use thesame definition of what a utility class is. The most significant change is, that classes with
main()methods areno longer considered utility classes by
UseUtilityClass.The goal is to ensure that rules report issues on the correct line and highlight only the relevant lines.
For example, instead of flagging an entire class declaration (including its body), we now generally report only
the class name. For more details, see [java] Single Line Warnings #730
and [java] Review reported locations of rules #3769. While this effort
is still ongoing, the following Java rules have been updated in this release:
AbstractClassWithoutAbstractMethodAbstractClassWithoutAnyMethodAtLeastOneConstructorAvoidDollarSignsAvoidCatchingGenericExceptionAvoidSynchronizedStatement(now reports only on synchronized keyword and not the whole synchronized block)ClassNamingConventionsClassWithOnlyPrivateConstructorsShouldBeFinalCommentDefaultAccessModifierCommentRequiredCouplingBetweenObjects(now reports only on class identifier and not whole compilation unit anymore)CyclomaticComplexityDataClassExcessiveImports(now reports only on imports and not the whole compilation unit anymore)ExcessiveParameterListExcessivePublicCountExhaustiveSwitchHasDefault(now reports only on switch keyword and not the whole switch block)GodClassImplicitFunctionalInterfaceJUnit5TestShouldBePackagePrivateLocalHomeNamingConventionLocalInterfaceSessionNamingConventionMissingSerialVersionUIDMissingStaticMethodInNonInstantiatableClassNcssCountNonExhaustiveSwitch(now reports only on switch keyword and not the whole switch block)NoPackagePublicMemberInNonPublicTypeShortClassNameSingleMethodSingletonSwitchDensity(now reports only on switch keyword and not the whole switch block)TestClassWithoutTestCasesTooFewBranchesForSwitch(now reports only on switch keyword and not the whole switch block)TooManyFields(now reports only on class identifier and not the whole class body anymore)TooManyMethods(now reports only on class identifier and not the whole class body anymore)TooManyStaticImports(now reports only on the first static import and not the whole compilation unit anymore)UnnecessaryModifierUseUtilityClassRenamed rules and properties
JUnitJupiterTestShouldBePackagePrivate(Java Best Practices) was renamed fromJUnit5TestShouldBePackagePrivate.junitJupiterTestPatternof ruleMethodNamingConventions(Java Code Style) was renamed fromjunit5TestPattern.The old names still work but are deprecated.
🐛️ Fixed Issues
🚨️ API Changes
Deprecations
FieldDeclarationsShouldBeAtStartOfClassRule#visitis an implementation detail ofFieldDeclarationsShouldBeAtStartOfClassRule. It will be removed in a later release.CyclomaticComplexityRule#visitTypeDeclis an implementation detail ofCyclomaticComplexityRule. It will be removed in a later release.SwitchDensityRule#visitSwitchLikeis an implementation detail ofSwitchDensityRule. It will be removed in a later release.PmdKotlinParser#PmdKotlinParserhas been deprecated.Use
KotlinLanguageModule#getInstance,createProcessor,servicesandgetParserinsteadto retrieve a correctly configured parser instance.
KotlinHandler#KotlinHandlerhas been deprecated.Use
getInstance,createProcessorandservicesinstead to access the LanguageVersionHandlerfor Kotlin.
KotlinInnerNode#getImageandKotlinInnerNode#hasImageEqualTohave been deprecated.They have not been used yet in Kotlin and the long-term plan is to remove these methods on each node.
Concrete nodes (subclasses of KotlinInnerNode) should provide a more specific attribute like
"getName" or "getIdentifier" instead and not rely on "getImage".
The same deprecation has been done for
KotlinTerminalNode.See #4787 for more information.
Experimental API
KotlinLanguageProperties#PARSE_TIMEOUT_SECONDSKotlinLanguageProperties#getParseTimeoutSecondsAttributeViewKtClassDeclarationAttributesKtClassParameterAttributesKtCompanionObjectAttributesKtFunctionDeclarationAttributesKtImportAliasAttributesKtImportHeaderAttributesKtVariableDeclarationAttributesHasModifiersHasSimpleIdentifierKotlinInnerNode#attributes.The attributes are also automatically exposed for XPath rules.
✨️ Merged pull requests
📦️ Dependency updates
📈️ Stats
Configuration
📅 Schedule: (UTC)
🚦 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.
This PR was generated by Mend Renovate. View the repository job log.