PMD - schedule #160
phipagAnnotations
10 errors and 1 warning
|
Avoid unused private methods such as 'log()'.:
examples/powertools-examples-core-utilities/sam/src/main/java/helloworld/App.java#L101
Unused Private Method detects when a private method is declared but is unused.
UnusedPrivateMethod (Priority: 1, Ruleset: Best Practices)
https://docs.pmd-code.org/pmd-doc-7.14.0/pmd_rules_java_bestpractices.html#unusedprivatemethod
|
|
Avoid unused private methods such as 'log()'.:
examples/powertools-examples-core-utilities/sam-graalvm/src/main/java/helloworld/App.java#L98
Unused Private Method detects when a private method is declared but is unused.
UnusedPrivateMethod (Priority: 1, Ruleset: Best Practices)
https://docs.pmd-code.org/pmd-doc-7.14.0/pmd_rules_java_bestpractices.html#unusedprivatemethod
|
|
Avoid unused private methods such as 'log()'.:
examples/powertools-examples-core-utilities/gradle/src/main/java/helloworld/App.java#L97
Unused Private Method detects when a private method is declared but is unused.
UnusedPrivateMethod (Priority: 1, Ruleset: Best Practices)
https://docs.pmd-code.org/pmd-doc-7.14.0/pmd_rules_java_bestpractices.html#unusedprivatemethod
|
|
This utility class has a non-private constructor:
examples/powertools-examples-core-utilities/cdk/infra/src/main/java/cdk/CdkApp.java#L20
For classes that only have static methods, consider making them utility classes.
Note that this doesn't apply to abstract classes, since their subclasses may
well include non-static methods. Also, if you want this class to be a utility class,
remember to add a private constructor to prevent instantiation.
(Note, that this use was known before PMD 5.1.0 as UseSingleton).
UseUtilityClass (Priority: 1, Ruleset: Design)
https://docs.pmd-code.org/pmd-doc-7.14.0/pmd_rules_java_design.html#useutilityclass
|
|
Avoid unused private methods such as 'log()'.:
examples/powertools-examples-core-utilities/cdk/app/src/main/java/helloworld/App.java#L94
Unused Private Method detects when a private method is declared but is unused.
UnusedPrivateMethod (Priority: 1, Ruleset: Best Practices)
https://docs.pmd-code.org/pmd-doc-7.14.0/pmd_rules_java_bestpractices.html#unusedprivatemethod
|
|
This utility class has a non-private constructor:
examples/powertools-examples-cloudformation/infra/cdk/src/main/java/com/myorg/PowertoolsExamplesCloudformationCdkApp.java#L6
For classes that only have static methods, consider making them utility classes.
Note that this doesn't apply to abstract classes, since their subclasses may
well include non-static methods. Also, if you want this class to be a utility class,
remember to add a private constructor to prevent instantiation.
(Note, that this use was known before PMD 5.1.0 as UseSingleton).
UseUtilityClass (Priority: 1, Ruleset: Design)
https://docs.pmd-code.org/pmd-doc-7.14.0/pmd_rules_java_design.html#useutilityclass
|
|
Document empty constructor:
examples/powertools-examples-batch/src/main/java/org/demo/batch/model/Product.java#L26
Uncommented Empty Constructor finds instances where a constructor does not
contain statements, but there is no comment. By explicitly commenting empty
constructors it is easier to distinguish between intentional (commented)
and unintentional empty constructors.
UncommentedEmptyConstructor (Priority: 1, Ruleset: Documentation)
https://docs.pmd-code.org/pmd-doc-7.14.0/pmd_rules_java_documentation.html#uncommentedemptyconstructor
|
|
Document empty constructor:
examples/powertools-examples-batch/src/main/java/org/demo/batch/model/DdbProduct.java#L29
Uncommented Empty Constructor finds instances where a constructor does not
contain statements, but there is no comment. By explicitly commenting empty
constructors it is easier to distinguish between intentional (commented)
and unintentional empty constructors.
UncommentedEmptyConstructor (Priority: 1, Ruleset: Documentation)
https://docs.pmd-code.org/pmd-doc-7.14.0/pmd_rules_java_documentation.html#uncommentedemptyconstructor
|
|
Avoid unused method parameters such as 'c'.:
examples/powertools-examples-batch/src/main/java/org/demo/batch/kinesis/KinesisBatchHandler.java#L29
Reports parameters of methods and constructors that are not referenced them in the method body.
Parameters whose name starts with `ignored` or `unused` are filtered out.
Removing unused formal parameters from public methods could cause a ripple effect through the code base.
Hence, by default, this rule only considers private methods. To include non-private methods, set the
`checkAll` property to `true`.
UnusedFormalParameter (Priority: 1, Ruleset: Best Practices)
https://docs.pmd-code.org/pmd-doc-7.14.0/pmd_rules_java_bestpractices.html#unusedformalparameter
|
|
Avoid unused method parameters such as 'context'.:
examples/powertools-examples-batch/src/main/java/org/demo/batch/dynamo/DynamoDBStreamBatchHandler.java#L28
Reports parameters of methods and constructors that are not referenced them in the method body.
Parameters whose name starts with `ignored` or `unused` are filtered out.
Removing unused formal parameters from public methods could cause a ripple effect through the code base.
Hence, by default, this rule only considers private methods. To include non-private methods, set the
`checkAll` property to `true`.
UnusedFormalParameter (Priority: 1, Ruleset: Best Practices)
https://docs.pmd-code.org/pmd-doc-7.14.0/pmd_rules_java_bestpractices.html#unusedformalparameter
|
|
pmd_analyse
Unsupported github action event 'schedule' - cannot determine modified files. All files will be analyzed.
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
PMD Report
Expired
|
13.5 KB |
sha256:8f62f7db617e6475a5571c20716962ec1ae298aed492900b7343af343032e7b9
|
|