Skip to content

Remove transformers>=5.0 uv overrides#483

Merged
JoelNiklaus merged 1 commit into
huggingface:mainfrom
JoelNiklaus:remove-transformers-overrides
Apr 20, 2026
Merged

Remove transformers>=5.0 uv overrides#483
JoelNiklaus merged 1 commit into
huggingface:mainfrom
JoelNiklaus:remove-transformers-overrides

Conversation

@JoelNiklaus

@JoelNiklaus JoelNiklaus commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Problem

PR #481 added transformers>=5.0 overrides in three places to work around vLLM 0.19.0 not yet supporting transformers v5. Now that vLLM 0.19.1 is out with transformers v5 support, these overrides are no longer needed and add unnecessary complexity.

Solution

Remove all three override locations:

  • uv-overrides.txt — deleted entirely
  • pyproject.toml — removed transformers>=5.0 from the [inference] extra and removed the [tool.uv] override-dependencies section
  • .github/workflows/pypi-release.yml — removed --overrides uv-overrides.txt flag from the test install step

Testing

  • Dependency resolution should work without overrides now that vLLM 0.19.1 supports transformers v5.

Made with Cursor


Note

Low Risk
Low risk: removes packaging/CI dependency overrides and a pinned optional dependency; main risk is release/test installs failing if upstream dependency constraints still require the override.

Overview
Removes the temporary transformers>=5.0 override workaround now that upstream support has caught up.

The release workflow no longer installs from TestPyPI with uv using --overrides uv-overrides.txt, the uv-overrides.txt override file is deleted, and pyproject.toml drops transformers>=5.0 from the inference extra and removes the [tool.uv] override-dependencies section.

Reviewed by Cursor Bugbot for commit fc442d7. Configure here.

vLLM 0.19.1 now supports transformers v5, so the workaround
added in huggingface#481 for vLLM 0.19.0 is no longer needed.

Made-with: Cursor
@JoelNiklaus JoelNiklaus requested a review from lewtun April 20, 2026 12:34

@lewtun lewtun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the upgrade, LGTM!

@JoelNiklaus JoelNiklaus merged commit 94e3b79 into huggingface:main Apr 20, 2026
6 checks passed
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