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
F adding component to watchdog #69
Conversation
…ces into F_refactoring_watchdog
…ces into F_adding_component_to_watchdog
…ces into F_adding_component_to_watchdog
new_log.type = log[1] | ||
system_status.logs.append(new_log) | ||
|
||
if 10 < len(self.watchdog_logs): |
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.
I think it is better to not have this magic constant number as the 10. Better to have a parameter to define it and be able to change it if we want to
sr_watchdog/package.xml
Outdated
<description>The sr_watchdog package</description> | ||
<author email="michal@shadowrobot.com">Michal Kramarczyk</author> | ||
<maintainer email="software@shadowrobot.com">Shadow Robot Software Team</maintainer> | ||
<license>GPL</license> |
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.
GPL v2
@@ -0,0 +1,86 @@ | |||
#!/usr/bin/env python | |||
# | |||
# Copyright (C) 2017, 2018 Shadow Robot Company Ltd - All Rights Reserved. |
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.
wrong license? should be public?
Proposed changes
Updating watchdog to handle components and multiple classes
Types of changes
What types of changes does your code introduce?
Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. This is a reminder of what we should look for before merging this code. I have: