Code Review 2020 from 4272 for InfiniteRecharge2020
Karaoke4272 opened this issue · comments
- Why is there a capital Constants and a lowercase constants folder? Somewhat confusing.
- Perhaps consider renaming AutoLineIntoTrench.java, AutoLineTrenchThree.java, and AutoLineIntoTrenchFive.java so that it shows the difference in function between the three.
- More comments in general please?
- Redundant code that could be put into one document (ramsete in auto)
- StealTwoEnemyTrench.java commented out
- Flywheel folder classes seems to never be called anywhere. Also, why is one of the documents named InfiniteRecharge.java? That doesn't say anything about its function or where it should be.
- Well organized/ I like your code for the limelight control.
Thanks for looking through, I'll have our incoming leadership and underclassmen look through this and make changes once our team begins meeting virtually.