Release – 2.0.0 #19
release.yml
on: workflow_dispatch
Annotations
10 errors and 2 warnings
|
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
|
|
quality
Unsupported github action event 'workflow_dispatch' - cannot determine modified files. All files will be analyzed.
|
|
quality
Cache not found for keys: setup-java-Linux-x64-maven-39acfd3e0ac86d1c106fcc3408936e74d300cff1bd4e15e7f84e6087b1b02139
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
source
Expired
|
2.83 MB |
sha256:1c5e744b6ebb673e0b1ea6d4d16f5ad03e6fc0d22d6468e33ad7c15ebb6ab129
|
|