Skip to content

Update dependency upper bounds - #1236

Merged
dsikka merged 3 commits into
mainfrom
updatedep
Oct 15, 2026
Merged

dsikka merged 3 commits into
mainfrom
updatedep

Conversation

@dhuangnm

Copy link
Copy Markdown
Collaborator

Purpose

Update dependency upper bounds to be ready for 0.9.0 release.

Tests

All tests.

Checklist

I have filled in:

  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan/results, such as providing test command and pasting the results.
  • (Optional) The necessary documentation update.
  • I (a human) have written or reviewed the code in this pr to the best of my ability.

Signed-off-by: Dan Huang <dahuang@redhat.com>
@coderabbitai

coderabbitai Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Repository: vllm-project/speculators/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 19479d6b-ceb2-4db4-a139-b9600326af7c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: vllm-project/speculators/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 0bb49a28-8e3c-45fe-b5ce-2ba385584ce7
📥 Commits

Reviewing files that changed from the base of the PR and between c5d4f9d and 099df7e.

📒 Files selected for processing: 1
  • pyproject.toml

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

📝 Walkthrough

The project configuration updates the upper version bounds for datasets, numpy, torch, tqdm, and transformers. The lower bounds remain unchanged.

Changes: Dependency version bounds

Layer / File(s) Summary
Dependency version constraints
pyproject.toml
The upper bounds for datasets, numpy, torch, and tqdm increase. The transformers upper bound changes from below 5.17.0 to 5.19.0 inclusive.

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 099df

The wider bounds allow newer releases of five dependencies; no specific behavior break is identified, so the PR appears ready to merge subject to normal CI.

Pre-merge checks: Passed 5
Passed Passed checks: 5 passed
Check name Status Explanation
Title check Passed The title clearly identifies the main change: updating dependency upper bounds. It is concise and specific.
Description check Passed The description explains that the dependency upper bounds are updated for the 0.9.0 release and states that tests were run. It is related to the changeset.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches: 💡 1
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
🛠️ Fix failing CI checks: 💡
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@mergify

mergify Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Merge Protections

🟢 Merge protection satisfied — ready to merge.

Show 1 satisfied protection

🟢 Require approval from approved reviewers list

All pull requests must have at least one approving review from a member of the approved reviewers list before merging.

  • any of:
    • approved-reviews-by = dsikka
    • approved-reviews-by = rahul-tuli
    • approved-reviews-by = fynnsu
    • approved-reviews-by = orestis-z
    • approved-reviews-by = shanjiaz

@mergify

mergify Bot commented Oct 14, 2026

Copy link
Copy Markdown

The quality checks have failed. Please run make style and make quality under
the root directory to address the lint failures. You will need to install the
dev optional install to get the required linting packages:
https://github.com/vllm-project/speculators/blob/main/CONTRIBUTING.md

Signed-off-by: Fynn Schmitt-Ulms <fschmitt@redhat.com>
@mergify mergify Bot removed the quality-failed label Oct 14, 2026
@rahul-tuli rahul-tuli added the ready This PR is ready for review label Oct 15, 2026
@dsikka
dsikka enabled auto-merge (squash) October 15, 2026 20:03
@dsikka
dsikka merged commit 1c800f2 into main Oct 15, 2026
11 checks passed
@dsikka
dsikka deleted the updatedep branch October 15, 2026 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready This PR is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants