-
Notifications
You must be signed in to change notification settings - Fork 14
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
Deploy and invoke interfaces #491
Conversation
079bbfe
to
b46e2b2
Compare
@leokondrashov: Any news on this PR? |
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.
Sorry, the review slipped through the cracks a bit.
a93eb14
to
08f6843
Compare
Signed-off-by: Lazar Cvetković <[email protected]>
Signed-off-by: Lazar Cvetković <[email protected]>
Signed-off-by: Lazar Cvetković <[email protected]>
08f6843
to
bdc4fd9
Compare
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, the only question is about zipkin, see below.
Don't forget to log all ARs mentioned in the comments.
Signed-off-by: Lazar Cvetković <[email protected]>
No description provided.