Merge pull request #152 from frc937/javadoc-cleanup

Clean up JavaDoc post-Heartland

src/main/java/frc/robot/Constants.java

@@ -162,6 +162,7 @@ public static class Intake {
     /** Current limit (in amps) for the intake motor(s) */
     public static final int INTAKE_MOTOR_CURRENT_LIMIT = 40;
 
+    /** Idle mode for the intake motors. (Either brake or coast) */
     public static final IdleMode INTAKE_MOTOR_IDLE_MODE = IdleMode.kBrake;
   }
 

src/main/java/frc/robot/RobotContainer.java

@@ -93,7 +93,7 @@ public class RobotContainer {
   /** Singleton instance of {@link PDP} for the whole robot. */
   public static PDP pdp = new PDP();
 
-  /* TODO: JAVADOC */
+  /** Singleton instance of the intake {@link Camera} for the whole robot. */
   public static Camera intakeCamera = new Camera(0);
 
   /*

src/main/java/frc/robot/subsystems/Camera.java

@@ -17,21 +17,28 @@
 import edu.wpi.first.wpilibj.shuffleboard.Shuffleboard;
 import edu.wpi.first.wpilibj2.command.SubsystemBase;
 
+/** Subsystem for a USB camera. */
 public class Camera extends SubsystemBase {
   private int port;
   private UsbCamera camera;
 
-  /** Creates a new Camera. */
+  /**
+   * Creates a new Camera.
+   *
+   * @param port The USB port number on the RoboRIO that the camera is plugged into. (This will
+   *     determine which /dev/videoX device the CameraServer tries to find).
+   */
   public Camera(int port) {
     this.port = port;
   }
 
+  /**
+   * Starts the camera that this subsystem represents and adds it to the Driver tab on Shuffleboard.
+   */
   public void startCamera() {
     camera = CameraServer.startAutomaticCapture(port);
     camera.setConnectionStrategy(ConnectionStrategy.kKeepOpen);
 
-    System.out.println("Camera started");
-
     Shuffleboard.getTab("Driver").add(camera).withSize(4, 4);
   }