[kylestach1678]

I quite like the general project structure, and most of the code is really nicely written. Very self explanatory. That said, the one thing I don't personally like about the code is the teleop.java file - while it may work for now as the code is fairly simple, often you will want to include automation of certain systems which adds complexity. At some point, if all of this is in one file it can get extremely cluttered with functions, making it difficult to tell what is doing what. Breaking subsystem - specific code into separate files also lets you reuse certain functionality that you wrote for teleop in an autonomous routine, which always makes things easier.

(Disclaimer - I only read the frc codebase as I am absolutely clueless about FTC code)

All in all, this is pretty good code.