Skip to content

Conversation

glbrntt
Copy link
Collaborator

@glbrntt glbrntt commented Dec 20, 2024

Motivation:

We've added back the reflection server; we should have an example of how
to use it.

Modifications:

  • Add a reflection server example
  • Split the echo service into a separate file in the echo example so that it can be symlinked from the reflection server

Result:

More examples

Motivation:

We've added back the reflection server; we should have an example of how
to use it.

Modifications:

Add a reflection server example

Result:

More examples
@glbrntt glbrntt added the semver/none No version bump required. label Dec 20, 2024
@glbrntt glbrntt marked this pull request as ready for review December 20, 2024 14:51
@glbrntt glbrntt requested a review from gjcairo December 20, 2024 14:52
Copy link
Collaborator

@gjcairo gjcairo left a comment

Choose a reason for hiding this comment

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

LGTM, just one very tiny comment. Thanks George!

@glbrntt glbrntt enabled auto-merge (squash) December 20, 2024 15:09
@glbrntt glbrntt merged commit 5be11cd into grpc:main Dec 20, 2024
31 of 33 checks passed
@glbrntt glbrntt deleted the v2/reflection branch December 20, 2024 15:33
@rnro rnro mentioned this pull request Sep 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver/none No version bump required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants