-
-
Notifications
You must be signed in to change notification settings - Fork 79
feat: JS dispatchers blog post #441
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
Conversation
4048cf5 to
e07a4d6
Compare
|
This should be ready for review. |
|
@andreaTP very sorry, looks like we forgot this PR... :< |
yada
left a comment
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.
Great job, @andreaTP and my apologies for the delay :<
Please see the changes requested and let's merge, publish and advertise this asap ;)
e122499 to
d697a49
Compare
|
@yada I think I addressed all the comments, please let me know if there is anything left. |
yada
left a comment
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.
Few pending changes in the review
|
d697a49 to
ab0de12
Compare
Signed-off-by: andreatp <andrea.peruffo1982@gmail.com>
ab0de12 to
21d8ce6
Compare
yada
left a comment
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.
LGTM with big thanks

Description
WIP, this blog demonstrates step-by-step how to exercise JavaScript Dispatchers in Microcks.
TODO: