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

chore(huggingface-hub): Fix chat completion stream type by alexander-alderman-webb · Pull Request #7070 · getsentry/sentry-python · GitHub

chore(huggingface-hub): Fix chat completion stream type - #7070

Merged
alexander-alderman-webb merged 7 commits into
masterfrom
webb/add-huggingface-hub-types
Aug 7, 2026
Merged

chore(huggingface-hub): Fix chat completion stream type#7070
alexander-alderman-webb merged 7 commits into
masterfrom
webb/add-huggingface-hub-types

Conversation

alexander-alderman-webb commented Aug 6, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Description

Add huggingface-hub to the uv typing group.
Cast iterator to the correct type.

The type for chat_completion() can be found here:

https://github.com/huggingface/huggingface_hub/blob/b4f4e92226642a4472945a174b9761b688a27dae/src/huggingface_hub/inference/_client.py#L559

Issues

Reminders

alexander-alderman-webb marked this pull request as ready for review August 6, 2026 15:07
alexander-alderman-webb requested a review from a team as a code owner August 6, 2026 15:07
alexander-alderman-webb changed the base branch from renovate/lock-file-maintenance to master August 6, 2026 15:11

github-actions Bot commented Aug 6, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Codecov Results 📊

96974 passed | ⏭️ 6239 skipped | Total: 103213 | Pass Rate: 93.96% | Execution Time: 346m 55s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +11522
Passed Tests 📈 +11368
Failed Tests
Skipped Tests 📈 +154

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2506 uncovered lines.
❌ Project coverage is 89.91%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
- Coverage    89.95%    89.91%    -0.04%
==========================================
  Files          193       193         —
  Lines        24946     24828      -118
  Branches      9000      8930       -70
==========================================
+ Hits         22439     22322      -117
- Misses        2507      2506        -1
- Partials      1435      1416       -19

Generated by Codecov Action

Copy link
Copy Markdown
Member

Looks like there's some test failures, will wait until they're 🟢 before taking a look

alexander-alderman-webb merged commit 65ec535 into master Aug 7, 2026
381 of 394 checks passed
alexander-alderman-webb deleted the webb/add-huggingface-hub-types branch August 7, 2026 07:11
mgaligniana pushed a commit to mgaligniana/sentry-python that referenced this pull request Aug 9, 2026
Add `huggingface-hub` to the typing dependency group and fix resulting mypy errors.
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.

3 participants


Back | FazBrowse Home | New Git URL