-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Alas, the time for a new neo era #183
Conversation
…axs :0 InSANE health and wellness, this guy needs a ambulance!1! 🤒
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.
a few things that seem to be missing, but correct me if I am misunderstanding
src/main/java/frc/robot/subsystems/limelight/LimelightHelpers.java
Outdated
Show resolved
Hide resolved
remove unused variables and rename IZone method to use proper caps
surely frank is just waiting to review this one as well teehee |
whoever gets to it first >:) because i know you are itching to review it |
I have been waiting for this since 9:01 PM yesterday |
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.
Everything looks really good here!! I think this is all ready to go!
src/main/java/frc/robot/subsystems/limelight/LimelightHelpers.java
Outdated
Show resolved
Hide resolved
sry I seeing comments and reviewing at the same time |
surely this won't have a breaking change when it gets tested on real sparks... oh no no no no no |
Absolutely not ....... |
Need I say more? This guy is safe!