In the constructor for each of your commands, you should pass the instance variable from CommandBase instead of the Subsystem class name:
Code:
public LiftForks() {
// Use requires() here to declare subsystem dependencies
// eg. requires(chassis);
requires(forkMotor); // or whatever you named the instance variable
}
instead of
Code:
public LiftForks() {
// Use requires() here to declare subsystem dependencies
// eg. requires(chassis);
requries(ForkMotor);
}
Another potential issue is that you don't appear to ever be stopping the motor. Even when the commands end, the motor will continue to spin (at least until the motor safety kicks in). One way to solve this issue would be to add a stop() method to your Subsystem, and then calling that method in each command's end() method.