Skip to content
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

Document protobufs and structs #2814

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

spacey-sooty
Copy link
Contributor

Resolves #2812

@PeterJohnson
Copy link
Member

PeterJohnson commented Oct 21, 2024

As currently written, this doesn't actually document why you would want to use them, how to use them, or how to create your own, which is the real crux of the issue. I would expect to see documentation of the pros/cons of each of these approaches (sendable, protobuf, struct), as well as code examples of how to use each, and code examples of how to create custom ones. This CD post could be used as a starting point for the "why" piece (and a piece of the "how" for built-in ones): https://www.chiefdelphi.com/t/introducing-monologue-annotation-based-telemetry-and-data-logging-for-java-teams/443917/5

@spacey-sooty spacey-sooty marked this pull request as draft October 21, 2024 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document NT protobuf/struct support
2 participants