-
Notifications
You must be signed in to change notification settings - Fork 60
refactoring of results and events types definitions #1716
Conversation
Signed-off-by: Kostiantyn Bushko <kostiantyn.bushko@yahoo.com>
Signed-off-by: Kostiantyn Bushko <kostiantyn.bushko@yahoo.com>
Signed-off-by: Kostiantyn Bushko <kostiantyn.bushko@yahoo.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.
Looks good! At first I was going to suggest capitalizing the utils
namespace, but I checked the Google style guide and it actually says namespaces should be lowercase. We've obviously not followed that before, but I'm happy to do so going forward.
src/libaktualizr/CMakeLists.txt
Outdated
../../include/libaktualizr/types.h | ||
../../include/libaktualizr/events.h | ||
../../include/libaktualizr/results.h | ||
../../include/libaktualizr/campaign.h) |
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.
Weird indenting change here.
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.
hmm, interesting, looks different in my local file
@@ -186,7 +186,7 @@ call. This will provision with the server if required. This must be called befor | |||
---- | |||
boost::signals2::connection Aktualizr::SetSignalHandler(std::function<void(shared_ptr<event::BaseEvent>)> &handler) | |||
---- | |||
Returns a signal connection object, which can be disconnected if desired. The events are defined in the https://github.com/advancedtelematic/aktualizr/blob/master/src/libaktualizr/primary/events.h[`primary/events.h`] header. | |||
Returns a signal connection object, which can be disconnected if desired. The events are defined in the https://github.com/advancedtelematic/aktualizr/blob/master/include/libaktualizr/events.h[`libaktualizr/events.h`] header. |
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.
Good catches!
Codecov Report
@@ Coverage Diff @@
## master #1716 +/- ##
==========================================
+ Coverage 76.81% 77.55% +0.73%
==========================================
Files 185 185
Lines 13341 13204 -137
==========================================
- Hits 10248 10240 -8
+ Misses 3093 2964 -129
Continue to review full report at Codecov.
|
Signed-off-by: Kostiantyn Bushko <kostiantyn.bushko@yahoo.com>
Signed-off-by: Kostiantyn Bushko kostiantyn.bushko@yahoo.com