feat: Expose otel subcommand and add user-facing documentation#5244
Merged
feat: Expose otel subcommand and add user-facing documentation#5244
Conversation
Contributor
|
💻 Deploy preview deleted (feat: Expose otel subcommand and add user-facing documentation). |
blewis12
force-pushed
the
expose-otel-subcommand-and-add-documentation
branch
2 times, most recently
from
64c4129 to
80ab153
Compare
blewis12
force-pushed
the
expose-otel-subcommand-and-add-documentation
branch
2 times, most recently
from
1cf74a5 to
ba32cc2
Compare
blewis12
force-pushed
the
expose-otel-subcommand-and-add-documentation
branch
from
ba32cc2 to
2a42c8b
Compare
Member
Author
|
@clayton-cornell since we are now introducing the concept of the OTel and Default Engines, i've tried to incorporate this terminology into the command docs. Is there anywhere else you'd think would make sense to adjust to reflect these new concepts? Or anywhere that would make sense to add some further explanation for this feature? |
thampiotr
reviewed
Jan 13, 2026
Contributor
thampiotr
left a comment
thampiotr
left a comment
There was a problem hiding this comment.
From code side of things this ready to merge! Need to only refine the docs :)
Co-authored-by: Piotr <17101802+thampiotr@users.noreply.github.com>
thampiotr
approved these changes
Jan 14, 2026
Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR will expose the
otelsubcommand for use, and adds documentation in the form of theotel.mdsubcommand file. We've already got developer-facing information in our contributing guide which addresses the build changes, and so this documentation mainly focuses on explaining what the functionality theotelcommand exposesQuestion for reviewers
Is there anywhere else I should be adding documentation? Scanning around the code it feels like this is it, but let me know if that's not the case