chore: add linting to workflow #754
Annotations
7 errors
Avoid reassigning parameters such as 'html':
plugin/src/main/java/io/snyk/eclipse/plugin/html/OssHtmlProvider.java#L17
Reassigning values to incoming parameters of a method or constructor is not recommended, as this can
make the code more difficult to understand. The code is often read with the assumption that parameter values
don't change and an assignment violates therefore the principle of least astonishment. This is especially a
problem if the parameter is documented e.g. in the method's javadoc and the new content differs from the original
documented content.
Use temporary local variables instead. This allows you to assign a new name, which makes the code better
understandable.
Note that this rule considers both methods and constructors. If there are multiple assignments for a formal
parameter, then only the first assignment is reported.
AvoidReassigningParameters (Priority: 2, Ruleset: Best Practices)
https://docs.pmd-code.org/pmd-doc-7.9.0/pmd_rules_java_bestpractices.html#avoidreassigningparameters
|
build (macos-latest)
Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
build (windows-latest)
The job was canceled because "macos-latest" failed.
|
build (windows-latest)
The operation was canceled.
|
build (ubuntu-latest)
The job was canceled because "macos-latest" failed.
|
Avoid reassigning parameters such as 'html':
plugin/src/main/java/io/snyk/eclipse/plugin/html/OssHtmlProvider.java#L17
Reassigning values to incoming parameters of a method or constructor is not recommended, as this can
make the code more difficult to understand. The code is often read with the assumption that parameter values
don't change and an assignment violates therefore the principle of least astonishment. This is especially a
problem if the parameter is documented e.g. in the method's javadoc and the new content differs from the original
documented content.
Use temporary local variables instead. This allows you to assign a new name, which makes the code better
understandable.
Note that this rule considers both methods and constructors. If there are multiple assignments for a formal
parameter, then only the first assignment is reported.
AvoidReassigningParameters (Priority: 2, Ruleset: Best Practices)
https://docs.pmd-code.org/pmd-doc-7.9.0/pmd_rules_java_bestpractices.html#avoidreassigningparameters
|
build (ubuntu-latest)
The operation was canceled.
|
Artifacts
Produced during runtime
Name | Size | |
---|---|---|
PMD Report
|
1013 Bytes |
|