-
Notifications
You must be signed in to change notification settings - Fork 9
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
Mode manager prevents redundant mode changes #67
Conversation
Before executing a requested mode change, i.e. applying the according changes, it is now checking whether the mode is already active. If so, no action is taken, but mode change service request is still returned with success. #60 Signed-off-by: Nordmann Arne (CR/ADT3) <arne.nordmann@de.bosch.com>
Signed-off-by: Nordmann Arne (CR/ADT3) <arne.nordmann@de.bosch.com>
Codecov Report
@@ Coverage Diff @@
## master #67 +/- ##
==========================================
- Coverage 27.67% 27.63% -0.04%
==========================================
Files 12 12
Lines 990 995 +5
Branches 710 714 +4
==========================================
+ Hits 274 275 +1
+ Misses 259 258 -1
- Partials 457 462 +5
Continue to review full report at Codecov.
|
Signed-off-by: Nordmann Arne (CR/ADT3) <arne.nordmann@de.bosch.com>
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.
Check one open comment.
Thanks for the review, Niels! |
Before executing a requested mode change, i.e. applying the according changes, it is now checking whether the mode is already active. If so, no action is taken, but mode change service request is still returned with success.
#60