refs #1210 Forward result of NorthService::sendToDispatcher() to the …#1211
Conversation
…() to the output of NorthService::operation(). Signed-off-by: Florent Peyrusse <florent.peyrusse@external.equans.com>
FlorentP42
force-pushed
the
1210-return-value-of-operations-not-forwarded-propelry-to-north-plugins
branch
from
71444ed to
ae9f3ab
Compare
|
@MarkRiddoch: Please let me know when this can be reviewed. |
…rded-propelry-to-north-plugins
|
Sorry, for some reason I did not get a notification that this was waiting. I've just checked and the unit tests seem to be running now, I did an "Update Branch" to force a rerun of the tests. |
|
Can you see any unit test result then? For me it looks like this: It does not appear like they are even trying to run... If you wtill want to merge like this it's fine by me... But I would like to have confirmation through the unit tests that this change did not have any unplanned impacts somewhere else that the unit tests could reveal. PS: If you didn't get any notification for the current PR, you might want to have a look at #1208 which is in the same state. |
|
Oh, now it seems to be passed, all right then... |
…rded-propelry-to-north-plugins
…rded-propelry-to-north-plugins
…rded-propelry-to-north-plugins
FlorentP42
deleted the
1210-return-value-of-operations-not-forwarded-propelry-to-north-plugins
branch
2 participants
…output of NorthService::operation().
Closes #1210