[Jared Russell]

At a quick glance, I don't see anything major that is incorrect.

Some tips:

* Generally in Java, all class names should be in camel case, e.g. ThisIsAClass. When I saw your "shift" command, I was at first confused because I did not think it was a class name.

* You should never re-use a class name as a variable name. Consider the following line in CommandBase.java:

public static transmission transmission = new transmission();

While it works technically, this is confusing! Better to use the camel case class name, and then lower case for the actual instance.

* You probably want another command to shift back down - currently once you have shifted to high, there is no way back to low!

* In your "shift" command, you check if you are finished based on whether or not the current value of the solenoid has changed. This will work, but just be aware that the solenoid value will read as changed instantaneously...there is no sensor or anything to tell you when it has actually changed gears (typically it happens very quickly anyhow).