Release – 2.1.0 #20
phipagrelease.yml
on: workflow_dispatch
Annotations
11 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
|
|
publish
Process completed with exit code 1.
|
|
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-e25d94a29c6eb351b17d11c8cba7895090c765080d50c1e08146ca183b6553b2
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
source
Expired
|
2.91 MB |
sha256:097402d3afabc40fdc80642461751af61d457a61a39ca8aabcc50ea836f0b4e7
|
|