-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add LKCoreBattery driver #183
Conversation
6d132f6
to
ad821b2
Compare
9572322
to
bc71960
Compare
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.
une première review :)
faut que tu rebases sur develop, tous les checks ne sont pas run en l'état. |
e15757a
to
9d9d455
Compare
9d9d455
to
bd14adf
Compare
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.
quelques changements
bd14adf
to
de01611
Compare
Codecov Report
@@ Coverage Diff @@
## develop #183 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 15 18 +3
Lines 213 232 +19
=========================================
+ Hits 213 232 +19
Continue to review full report at Codecov.
|
de01611
to
2f94fe7
Compare
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.
nouvelle review :) on s'approche doucement ! lis bien tous les commentaires, j'ai bien expliqué, j'ai commencé les modifications mais j'ai pas tout fait, à toi de creuser.
n'hésite pas à prendre du recul, regarde les fonctions de loin, cherche les symétries ou l'absence de symétrie --> c'est mieux d'en avoir si c'est la même chose.
ba9115e
to
9027b1f
Compare
7d2e7b2
to
9a393c5
Compare
The spike is a test to mesure how much time it takes for the battery to completely discharge while running a standard behavioral program Co-authored-by: Ladislas de Toldi <ladislas@detoldi.me>
0cfc37b
to
5426877
Compare
Code Climate has analyzed commit 5426877 and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 100.0% (50% is the threshold). This pull request will bring the total coverage in the repository to 100.0% (0.0% change). View more on Code Climate. |
Just took what have been done with the microphone and replace some names so it can work with the battery AnalogIn.