Skip to content

Merge pull request #10 from garrettsummerfi3ld/qodana-automation-9321… #74

Merge pull request #10 from garrettsummerfi3ld/qodana-automation-9321…

Merge pull request #10 from garrettsummerfi3ld/qodana-automation-9321… #74

Triggered via push October 13, 2024 04:49
Status Failure
Total duration 2m 9s
Artifacts

codeql.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

1 error, 23 warnings, and 1 notice
CodeQL-Build
We were unable to automatically build your code. Please replace the call to the autobuild action with your custom build steps. Encountered a fatal error while running "/opt/hostedtoolcache/CodeQL/2.19.1/x64/codeql/java/tools/autobuild.sh". Exit code was 1 and last log line was: Picked up JAVA_TOOL_OPTIONS: -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false. See the logs for more details.
Nullability and data flow problems: src/main/java/frc/robot/Robot.java#L79
Switch label `0` is unreachable
Nullability and data flow problems: src/main/java/frc/robot/Robot.java#L82
Switch label `1` is unreachable
Field can be local: src/main/java/frc/robot/subsystems/Vision.java#L29
Field can be converted to a local variable
Field can be local: src/main/java/frc/robot/subsystems/Vision.java#L28
Field can be converted to a local variable
Field may be 'final': src/main/java/frc/robot/util/Alert.java#L21
Field `groups` may be 'final'
Inner class may be 'static': src/main/java/frc/robot/Constants.java#L158
Inner class `Limelight` may be 'static'
Inner class may be 'static': src/main/java/frc/robot/Constants.java#L33
Inner class `HardwareConstants` may be 'static'
Inner class may be 'static': src/main/java/frc/robot/Constants.java#L178
Inner class `ConveyorCamera` may be 'static'
Inner class may be 'static': src/main/java/frc/robot/Constants.java#L168
Inner class `IntakeCamera` may be 'static'
Non-final field in 'enum': src/main/java/frc/robot/subsystems/ConveyorSubsystem.java#L120
Non-final field `speed` in enum 'FlywheelSpeed'
Call to 'printStackTrace()': src/main/java/frc/robot/Robot.java#L113
Call to `printStackTrace()` should probably be replaced with more robust logging
'Collection.toArray()' call style: src/main/java/frc/robot/Robot.java#L180
Call to `toArray()` with pre-sized array argument 'new String\[clientAddresses.size()\]'
'Collection.toArray()' call style: src/main/java/frc/robot/Robot.java#L178
Call to `toArray()` with pre-sized array argument 'new String\[clientNames.size()\]'
Redundant 'if' statement: src/main/java/frc/robot/Robot.java#L164
`if` statement can be simplified
Redundant 'if' statement: src/main/java/frc/robot/subsystems/Vision.java#L144
`if` statement can be simplified
Unnecessary modifier: src/main/java/frc/robot/util/Alert.java#L109
Modifier `static` is redundant for inner enums