Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Added missing step causing compilation issue #20000

Merged
merged 3 commits into from
Sep 28, 2020
Merged

Conversation

cpoDesign
Copy link
Contributor

Fixed guide to remove compilation issue caused by incorrect namespace in greet.proto.

Fixed guide to remove compilation issue caused by incorrect namespace in greet.proto.
@guardrex guardrex requested a review from JunTaoLuo September 25, 2020 00:18
@cpoDesign
Copy link
Contributor Author

JunTaoLuo @guardrex Is there anything else you need me to do? from my perspective, it is completed

@guardrex
Copy link
Collaborator

We just need John to approve this and then we can get it merged. Stand-by for a bit longer ... the product unit 🐈 is very busy these days with the upcoming .NET 5 release, so there will be short delays for doc reviews during this period. 🏃🏃🏃🏃🏃😅

Copy link
Contributor

@JunTaoLuo JunTaoLuo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good. We updated our server templates to include adjusting the namespace but we never updated the docs for creating the client.

@guardrex guardrex merged commit e655869 into dotnet:master Sep 28, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants