-
Notifications
You must be signed in to change notification settings - Fork 268
Add reasoning content message part #2797
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
Merged
lmolkova
merged 5 commits into
open-telemetry:main
from
Cirilla-zmh:minghui/reasoning_content
Oct 1, 2025
Merged
Add reasoning content message part #2797
lmolkova
merged 5 commits into
open-telemetry:main
from
Cirilla-zmh:minghui/reasoning_content
Oct 1, 2025
Conversation
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
alexmojaki
reviewed
Sep 22, 2025
lmolkova
reviewed
Sep 23, 2025
Change-Id: I0402112138f95152f89b95a9bb6c3feddd3c1d35 Co-developed-by: Cursor <[email protected]> Change-Id: I05ba8af71980dd9d86e41544b9a75901c30f7d3c Co-developed-by: Cursor <[email protected]> Change-Id: I9bec00a3fdc7c3179c740520a56095f729d55567 Co-developed-by: Cursor <[email protected]>
8829b82
to
c9410b8
Compare
Change-Id: I693ed9b16739e3763a5a6eff66da29cc6cf21123 Co-developed-by: Cursor <[email protected]>
Change-Id: I11c1b5b86a0aaefe196b5fc17dd633f4dbd9b97c Co-developed-by: Cursor <[email protected]> Change-Id: I7790ff9436ff564f092202f14ae4b07aca4b0ce9 Co-developed-by: Cursor <[email protected]> Change-Id: I722323321ae226259e3dcaf32576ba23f8ba66fd Co-developed-by: Cursor <[email protected]>
2e66461
to
cded476
Compare
alexmojaki
reviewed
Sep 25, 2025
alexmojaki
approved these changes
Sep 26, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
@lmolkova I believe this PR could be merged. Could you please check it again? |
steverao
approved these changes
Sep 30, 2025
lmolkova
approved these changes
Oct 1, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Part of #1965
Changes
Add reasoning content message part.
Refer:
Note: if the PR is touching an area that is not listed in the existing areas, or the area does not have sufficient domain experts coverage, the PR might be tagged as experts needed and move slowly until experts are identified.
Merge requirement checklist
[chore]