update node docs to use addService

grpc / grpc.github.io

The grpc.io website. (on GitHub pages)

276 stars 381 forks source link

update node docs to use addService #776

Open mitchdraft opened 5 years ago

mitchdraft commented 5 years ago

small change to the docs now that the node server api has changed

link to api: https://github.com/grpc/grpc-node/blob/master/packages/grpc-native-core/src/server.js#L948

the examples work with or without this change, but the change removes this warning message:

$ node server.js
(node:44083) DeprecationWarning: Server#addProtoService: Use Server#addService instead

thelinuxfoundation commented 5 years ago

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.

After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

User @mitchdraft isn't covered by a CLA. They will need to complete the form at https://identity.linuxfoundation.org/projects/cncf

Regards, CLA GitHub bot

mitchdraft commented 5 years ago

CLA signed

ZhouyihaiDing commented 5 years ago

The change seems fine to me too. @murgatroid99 can you please take a look?

hsaliak commented 5 years ago

@murgatroid99 @nicolasnoble can you please rubber stamp, given that this is an example change it would be best if one of you looked over it.

thisisnotapril commented 5 years ago

@murgatroid99 @nicolasnoble @hsaliak are we OK to merge this one?