-
Notifications
You must be signed in to change notification settings - Fork 206
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Examples Clean-Up #1408
Examples Clean-Up #1408
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When I leave a review, I say "LGTM" because the code does "look good to me." Very demure, very mindful.
Before merge, I want to take a read through here haha. Will get done Monday or Tuesday this week. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
appreciate all the hard work on this, much needed
photonlib-java-examples/aimandrange/src/main/java/frc/robot/Robot.java
Outdated
Show resolved
Hide resolved
#1418 needs to happen shortly after this is merged. |
…lllllllllllleeeeeeeeeeeeeeeesssssssssssssssssssssssssssssssssssssssssssssssssssss
Updates address all my comments, ty! |
Goals:
Minimize content to achieve those goals.