-
-
Notifications
You must be signed in to change notification settings - Fork 524
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
Extend error response #1022
Extend error response #1022
Conversation
qazwsxedckll
commented
Jan 19, 2024
- add ErrorReason in proto file.
- return in service
- assert error
1. fix generating duplicate respond function when multiple services in one proto file. 2. update all examples to use new generator. 3. fix opentelemetry-custom-provider
@qazwsxedckll Why was the directory name |
@qazwsxedckll Tracking the |
👋🏻 @cupen |
@cupen Maybe there were many problems before go.work came out so add underscore to ignore them make sense. Now we can just run I have no opinions on |
Hi @rogeralsing @qazwsxedckll To my point of view, these examples are not a part of So, for avoid the lots of unnecessary changes with btw: These examples also could be the first choice to check our locally modification, so the |
@cupen First, |
@qazwsxedckll Yes, that's true. I mistakenly thought you wanted to put a |