FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat(streaming): Make `set_conversation_id` API work by sentrivana · Pull Request #6866 · getsentry/sentry-python · GitHub

feat(streaming): Make set_conversation_id API work - #6866

Merged
sentrivana merged 2 commits into
masterfrom
ivana/streaming-conversation-id
Jul 24, 2026
Merged

feat(streaming): Make set_conversation_id API work#6866
sentrivana merged 2 commits into
masterfrom
ivana/streaming-conversation-id

Conversation

sentrivana commented Jul 23, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Description

In streaming mode, the set_conversation_id API doesn't currently do anything. Change that.

Issues

Reminders

Copy link
Copy Markdown
Contributor

Codecov Results 📊

95450 passed | ⏭️ 6340 skipped | Total: 101790 | Pass Rate: 93.77% | Execution Time: 336m 24s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +135
Passed Tests 📈 +135
Failed Tests
Skipped Tests

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2510 uncovered lines.
✅ Project coverage is 89.68%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.65%    89.68%    +0.03%
==========================================
  Files          193       193         —
  Lines        24324     24331        +7
  Branches      8572      8576        +4
==========================================
+ Hits         21807     21821       +14
- Misses        2517      2510        -7
- Partials      1395      1395         —

Generated by Codecov Action

sentrivana marked this pull request as ready for review July 24, 2026 07:39
sentrivana requested a review from a team as a code owner July 24, 2026 07:39
sentrivana merged commit d3ca0b1 into master Jul 24, 2026
146 checks passed
sentrivana deleted the ivana/streaming-conversation-id branch July 24, 2026 09:16
mgaligniana pushed a commit to mgaligniana/sentry-python that referenced this pull request Aug 9, 2026
### Description
In streaming mode, the `set_conversation_id` API doesn't currently do
anything. Change that.

#### Issues
<!--
* resolves: getsentry#1234
* resolves: LIN-1234
-->

#### Reminders
- Please add tests to validate your changes, and lint your code using
`uv run ruff`.
- Add GH Issue ID _&_ Linear ID (if applicable)
- PR title should use [conventional
commit](https://develop.sentry.dev/engineering-practices/commit-messages/#type)
style (`feat:`, `fix:`, `ref:`, `meta:`)
- For external contributors:
[CONTRIBUTING.md](https://github.com/getsentry/sentry-python/blob/master/CONTRIBUTING.md),
[Sentry SDK development docs](https://develop.sentry.dev/sdk/), [Discord
community](https://discord.gg/Ww9hbqr)
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 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.

2 participants


Back | FazBrowse Home | New Git URL