Skip to content

Merge pull request #38 from OakvilleDynamics/event/stlouis-regional #171

Merge pull request #38 from OakvilleDynamics/event/stlouis-regional

Merge pull request #38 from OakvilleDynamics/event/stlouis-regional #171

Triggered via push March 23, 2024 21:14
Status Success
Total duration 1m 43s
Artifacts

qodana.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

11 warnings
qodana
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: JetBrains/qodana-action@v2023.3.1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Nullability and data flow problems: src/main/java/frc/robot/Robot.java#L66
Switch label `0` is unreachable
Nullability and data flow problems: src/main/java/frc/robot/Robot.java#L69
Switch label `1` is unreachable
Statement with empty body: src/main/java/frc/robot/util/Time.java#L16
`while` statement has empty body
Inner class may be 'static': src/main/java/frc/robot/Constants.java#L119
Inner class `ConveyorCamera` may be 'static'
Inner class may be 'static': src/main/java/frc/robot/Constants.java#L109
Inner class `IntakeCamera` may be 'static'
Inner class may be 'static': src/main/java/frc/robot/Constants.java#L99
Inner class `LimeLight` may be 'static'
Inner class may be 'static': src/main/java/frc/robot/Constants.java#L98
Inner class `VisionConstants` may be 'static'
Simplifiable conditional expression: src/main/java/frc/robot/subsystems/swervedrive/SwerveSubsystem.java#L128
`alliance.isPresent() ? alliance.get() == DriverStation.Alliance.Red : false` can be simplified to 'alliance.isPresent() \&\& alliance.get() == DriverStation.Alliance.Red'
Suspicious variable/parameter name combination: src/main/java/frc/robot/subsystems/Vision.java#L45
'WIDTH' should probably not be passed as parameter 'y'
Suspicious variable/parameter name combination: src/main/java/frc/robot/subsystems/Vision.java#L31
'WIDTH' should probably not be passed as parameter 'y'