figured it out 🤦 #609
main.yml
on: push
Linting
12s
Spell Check
9s
Find merge conflicts
3s
Test Javadocs Generation
1m 17s
Build
1m 28s
Annotations
2 errors and 15 warnings
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.com/orgs/community/discussions/26680#discussioncomment-3252835
|
Test Javadocs Generation
Process completed with exit code 1.
|
Find merge conflicts
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
Spell Check
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
Linting
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
Linting:
src/main/java/frc/robot/Main.java#L20
[checkstyle] reported by reviewdog 🐶
<p> tag should be placed immediately before the first word, with no space after.
Raw Output:
/github/workspace/./src/main/java/frc/robot/Main.java:20:0: warning: <p> tag should be placed immediately before the first word, with no space after. (com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck)
|
Linting:
src/main/java/frc/robot/RobotContainer.java#L147
[checkstyle] reported by reviewdog 🐶
Comment has incorrect indentation level 12, expected is 20, indentation should be the same level as line 143.
Raw Output:
/github/workspace/./src/main/java/frc/robot/RobotContainer.java:147:13: warning: Comment has incorrect indentation level 12, expected is 20, indentation should be the same level as line 143. (com.puppycrawl.tools.checkstyle.checks.indentation.CommentsIndentationCheck)
|
Linting:
src/main/java/frc/robot/Robot.java#L125
[checkstyle] reported by reviewdog 🐶
<p> tag should be placed immediately before the first word, with no space after.
Raw Output:
/github/workspace/./src/main/java/frc/robot/Robot.java:125:0: warning: <p> tag should be placed immediately before the first word, with no space after. (com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck)
|
|
|
|
|
|
Linting:
src/main/java/frc/robot/commands/customproxy/ProxyConditionalCommand.java#L8
[checkstyle] reported by reviewdog 🐶
Missing a Javadoc comment.
Raw Output:
/github/workspace/./src/main/java/frc/robot/commands/customproxy/ProxyConditionalCommand.java:8:1: warning: Missing a Javadoc comment. (com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocTypeCheck)
|
Linting:
src/main/java/frc/robot/commands/customproxy/ProxyParallelCommand.java#L9
[checkstyle] reported by reviewdog 🐶
Missing a Javadoc comment.
Raw Output:
/github/workspace/./src/main/java/frc/robot/commands/customproxy/ProxyParallelCommand.java:9:1: warning: Missing a Javadoc comment. (com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocTypeCheck)
|
Test Javadocs Generation
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
Build
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|