Conversation
|
If you run |
Thank you for the guidance! Running My node version is Does this PR depend on unit tests to be "approvable"? I can add the test code and someone else can try to run them? |
|
I think the most likely issue you are seeing is that the git submodules that the setup script expects to be able to use are absent. You can correct that by running |
Thank you!, second approach worked (even tho 29 tests fail). I will work on the tests shortly. |
bb60aee to
3b900f7
Compare
3b900f7 to
2fb705e
Compare
|
Hey @murgatroid99 can you help me understand why Windows Tests job is failing? doesn't seem to be related to my changes I believe. |
|
Ignore the Windows test job. It's broken. |
murgatroid99
left a comment
There was a problem hiding this comment.
Thank you for adding this.
|
Way to go my friend! Thanks for the help! |
|
This has been released in version 2.1.0. |
Change
This PR introduces the implementation of the new
Listendpoint to theHealthservice in order to list all the services that a certain server is watching to.It also adds generated files for the endpoint and its tests.
I would like to write some tests too but I couldn't run the test suite as it is as I get some errors, any help is appreciated.
Related PR
gRPC-proto added in #2951
Proposal
grpc/proposal#468