-
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
Change Logger to start transition to Sendables #11
base: master
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## master #11 +/- ##
============================================
- Coverage 17.78% 16.33% -1.46%
Complexity 17 17
============================================
Files 13 13
Lines 371 404 +33
Branches 25 32 +7
============================================
Hits 66 66
- Misses 305 338 +33
|
Sent from my iPad
CodeClimate is just complaining about Javadoc formatting, not using an external logging library and code duplication that @tricktrap and I have decided would leave the code clearer to ignore. I will fix the Javadocs later tonight, but other than that, I do believe that this is ready for review. |
Code Climate has analyzed commit fbbc581 and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 0.0% (50% is the threshold). This pull request will bring the total coverage in the repository to 16.3% (-1.3% change). View more on Code Climate. |
WPILib is now including a much easier way to have classes interact with SmartDashboard. While there is no on-bot logging (yet), having WPILib handle the outputting of variables to SmartDashboard and having the ability to change values from the driver station on-the-fly might be beneficial.