refactor: Add new methods according to new approach of keploy serve #295
gouravkrosxmaven.yml
on: pull_request
build
39s
checkstyle linting
14s
Annotations
10 errors and 1 warning
|
checkstyle linting:
v2/src/main/java/io.keploy.cli/KeployCLI.java#L9
[checkstyle] reported by reviewdog 🐶
Unused import - java.io.FileInputStream.
Raw Output:
/github/workspace/./v2/src/main/java/io.keploy.cli/KeployCLI.java:9:8: error: Unused import - java.io.FileInputStream. (com.puppycrawl.tools.checkstyle.checks.imports.UnusedImportsCheck)
|
|
checkstyle linting:
v2/src/main/java/io.keploy.cli/KeployCLI.java#L10
[checkstyle] reported by reviewdog 🐶
Unused import - java.io.FileOutputStream.
Raw Output:
/github/workspace/./v2/src/main/java/io.keploy.cli/KeployCLI.java:10:8: error: Unused import - java.io.FileOutputStream. (com.puppycrawl.tools.checkstyle.checks.imports.UnusedImportsCheck)
|
|
checkstyle linting:
v2/src/main/java/io.keploy.cli/KeployCLI.java#L19
[checkstyle] reported by reviewdog 🐶
Unused import - java.nio.file.Paths.
Raw Output:
/github/workspace/./v2/src/main/java/io.keploy.cli/KeployCLI.java:19:8: error: Unused import - java.nio.file.Paths. (com.puppycrawl.tools.checkstyle.checks.imports.UnusedImportsCheck)
|
|
checkstyle linting:
v2/src/main/java/io.keploy.cli/KeployCLI.java#L26
[checkstyle] reported by reviewdog 🐶
Line is longer than 80 characters (found 112).
Raw Output:
/github/workspace/./v2/src/main/java/io.keploy.cli/KeployCLI.java:26:0: error: Line is longer than 80 characters (found 112). (com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck)
|
|
checkstyle linting:
v2/src/main/java/io.keploy.cli/KeployCLI.java#L38
[checkstyle] reported by reviewdog 🐶
Missing a Javadoc comment.
Raw Output:
/github/workspace/./v2/src/main/java/io.keploy.cli/KeployCLI.java:38:5: error: Missing a Javadoc comment. (com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocVariableCheck)
|
|
checkstyle linting:
v2/src/main/java/io.keploy.cli/KeployCLI.java#L40
[checkstyle] reported by reviewdog 🐶
Missing a Javadoc comment.
Raw Output:
/github/workspace/./v2/src/main/java/io.keploy.cli/KeployCLI.java:40:5: error: Missing a Javadoc comment. (com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocVariableCheck)
|
|
checkstyle linting:
v2/src/main/java/io.keploy.cli/KeployCLI.java#L42
[checkstyle] reported by reviewdog 🐶
Missing a Javadoc comment.
Raw Output:
/github/workspace/./v2/src/main/java/io.keploy.cli/KeployCLI.java:42:5: error: Missing a Javadoc comment. (com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocVariableCheck)
|
|
checkstyle linting:
v2/src/main/java/io.keploy.cli/KeployCLI.java#L70
[checkstyle] reported by reviewdog 🐶
Missing a Javadoc comment.
Raw Output:
/github/workspace/./v2/src/main/java/io.keploy.cli/KeployCLI.java:70:5: error: Missing a Javadoc comment. (com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck)
|
|
checkstyle linting:
v2/src/main/java/io.keploy.cli/KeployCLI.java#L70
[checkstyle] reported by reviewdog 🐶
Name 'StartUserApplication' must match pattern '^[a-z][a-zA-Z0-9]*$'.
Raw Output:
/github/workspace/./v2/src/main/java/io.keploy.cli/KeployCLI.java:70:24: error: Name 'StartUserApplication' must match pattern '^[a-z][a-zA-Z0-9]*$'. (com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck)
|
|
checkstyle linting
reviewdog: Too many results (annotations) in diff.
You may miss some annotations due to GitHub limitation for annotation created by logging command.
Please check GitHub Actions log console to see all results.
Limitation:
- 10 warning annotations and 10 error annotations per step
- 50 annotations per job (sum of annotations from all the steps)
- 50 annotations per run (separate from the job annotations, these annotations aren't created by users)
Source: https://github.community/t5/GitHub-Actions/Maximum-number-of-annotations-that-can-be-created-using-GitHub/m-p/39085
|
|
checkstyle linting
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|