Skip to content

fix(huggingface_hub): Stop setting transaction status in huggingface_hub#5952

Closed
Zenithatic wants to merge 1 commit intogetsentry:masterfrom
Zenithatic:zenithatic/huggingfacehub/stop-set-transaction-status
Closed

fix(huggingface_hub): Stop setting transaction status in huggingface_hub#5952
Zenithatic wants to merge 1 commit intogetsentry:masterfrom
Zenithatic:zenithatic/huggingfacehub/stop-set-transaction-status

Conversation

@Zenithatic
Copy link
Copy Markdown

Description

  • Prevent the Hugging Face AI integration from changing root HTTP transaction status when API call fails

  • Keep error visibility on the integration span so failures are still captured and observable

Issues

Reminders

@sdk-maintainer-bot sdk-maintainer-bot bot added missing-maintainer-discussion Used for automated community contribution checks. violating-contribution-guidelines Used for automated community contribution checks. labels Apr 7, 2026
@sdk-maintainer-bot
Copy link
Copy Markdown

This PR has been automatically closed. The referenced issue does not show a discussion between you and a maintainer.

To avoid wasted effort on both sides, please discuss your proposed approach in the issue first and wait for a maintainer to respond before opening a PR.

Please review our contributing guidelines for more details.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • (integrations) Instrument pyreqwest tracing by servusdei2018 in #5682

Bug Fixes 🐛

  • (huggingface_hub) Stop setting transaction status in huggingface_hub by Zenithatic in #5952

Internal Changes 🔧

  • (openai) Split token counting by API for easier deprecation by ericapisani in #5930
  • (opentelemetry) Ignore mypy error by alexander-alderman-webb in #5927
  • Fix license metadata in setup.py by sl0thentr0py in #5934
  • Update validate-pr workflow by stephanie-anderson in #5931

🤖 This preview updates automatically when you update the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

missing-maintainer-discussion Used for automated community contribution checks. violating-contribution-guidelines Used for automated community contribution checks.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

huggingface_hub: Stop setting transaction status

1 participant