-
Notifications
You must be signed in to change notification settings - Fork 207
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
E1.33 cherry pick part 4 #1950
E1.33 cherry pick part 4 #1950
Conversation
(cherry picked from commit bdb40db)
(cherry picked from commit 847734f)
…heartbeat behaviour and timing (cherry picked from commit 4d7049c)
(cherry picked from commit d1eb7b8)
(cherry picked from commit 669e2e5)
(cherry picked from commit ec58616)
(cherry picked from commit 40ff84c)
(cherry picked from commit b3942cf)
(cherry picked from commit 218c4c7)
(cherry picked from commit e00322d)
(cherry picked from commit 16455b1)
Spelling will be fixed once #1952 is merged in. |
Done :) |
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.
See inline comments :)
(cherry picked from commit 91e014f)
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.
Nice that you added tests for the methods when NULL is passed as input 👍
Yeah I like a bit of test driven development; make it segfault, then stop it doing so! |
Okay I think that's the outstanding comments fixed in one way or another @kripton . |
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.
Re-reviewed ✔️
b93b8e0
into
OpenLightingProject:master
No description provided.