Change Logger to start transition to Sendables#11
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ 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.