Skip to content
Qodana

Merge pull request #39 from OakvilleDynamics/dependabot/github_action… #175

Sign in to view logs

Merge pull request #39 from OakvilleDynamics/dependabot/github_action…

Merge pull request #39 from OakvilleDynamics/dependabot/github_action… #175

Triggered via push March 27, 2024 06:59
@dependabotdependabot[bot]
pushed 827be20
main
Status Success
Total duration 1m 44s
Artifacts –

qodana.yml

on: push
qodana
1m 33s
qodana
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
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'