-
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
ladislas+hugo/feature/quit activity mode #1062
Conversation
ladislas
commented
Oct 13, 2022
•
edited by HPezz
Loading
edited by HPezz
- Validated on robot
File comparision analysis report🔖 Info
Click to show memory sections
📝 SummaryClick to show summary
🗺️ Map files diff outputClick to show diff list
|
File comparision analysis report🔖 Info
Click to show memory sections
📝 SummaryClick to show summary
🗺️ Map files diff outputClick to show diff list
|
7b36dc1
to
e6076a8
Compare
Codecov Report
@@ Coverage Diff @@
## hugo/feature/Add-operator-+=-to-kernel-stub #1062 +/- ##
===============================================================================
+ Coverage 96.00% 96.05% +0.04%
===============================================================================
Files 133 133
Lines 3178 3190 +12
===============================================================================
+ Hits 3051 3064 +13
+ Misses 127 126 -1
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
1e6cee9
to
df54d6a
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.
LGTM 👍
I made a suggestion regarding the conditions to make the workflow clearer.
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.
- Validated on the robot
A case is not covered or all cases are not clearly specified.
Some question about tests
libs/RobotKit/tests/RobotController_test_stateAutonomousActivities.cpp
Outdated
Show resolved
Hide resolved
libs/RobotKit/tests/RobotController_test_stateAutonomousActivities.cpp
Outdated
Show resolved
Hide resolved
libs/RobotKit/tests/RobotController_test_stateAutonomousActivities.cpp
Outdated
Show resolved
Hide resolved
dec0bb6
to
a587177
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.
- Partially validated on robot:
Idle
/Working
↔Autonomous Activities
(Activity not playing)
Some suggestions
Some code is not covered according to sonarcloud, but no clue on how to reach them considering preceding conditions
libs/RobotKit/tests/RobotController_test_stateAutonomousActivities.cpp
Outdated
Show resolved
Hide resolved
a587177
to
9de637f
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.
LGTM 👍
9de637f
to
6266ca8
Compare
52bcc6d
to
f34e582
Compare
6266ca8
to
a40c6ad
Compare
Kudos, SonarCloud Quality Gate passed! |