-
Notifications
You must be signed in to change notification settings - Fork 130
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
feat(provider-sdk): add run_provider_handler
#1652
feat(provider-sdk): add run_provider_handler
#1652
Conversation
7888c4a
to
9d837de
Compare
init_provider
run_provider_handler
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.
This all looks fine to me, @vados-cosmonic deferring to you on review for anything in the bindgen macro etc etc
@brooksmtownsend will do. This is going to break my other PRs but let's get this merged in first since it's a good refactor of |
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, nothing but nits 🚀
Signed-off-by: Roman Volosatovs <rvolosatovs@riseup.net>
Signed-off-by: Roman Volosatovs <rvolosatovs@riseup.net>
Signed-off-by: Roman Volosatovs <rvolosatovs@riseup.net>
Signed-off-by: Roman Volosatovs <rvolosatovs@riseup.net>
Signed-off-by: Roman Volosatovs <rvolosatovs@riseup.net>
Signed-off-by: Roman Volosatovs <rvolosatovs@riseup.net>
Signed-off-by: Roman Volosatovs <rvolosatovs@riseup.net>
Signed-off-by: Roman Volosatovs <rvolosatovs@riseup.net>
674f5da
to
364ccd0
Compare
Signed-off-by: Roman Volosatovs <rvolosatovs@riseup.net>
364ccd0
to
5e33655
Compare
Signed-off-by: Roman Volosatovs <rvolosatovs@riseup.net>
Feature or Problem
run_provider_handler
for building providers using justProviderHandler
trait implementations, which manually serve using returnedwrpc_transport::Client
(e.g. all handwritten providers)Related Issues
Release Information
Consumer Impact
Testing
Unit Test(s)
Acceptance or Integration
Manual Verification