Skip to content

Edit dependency groups#7257

Merged
youknowone merged 2 commits intoRustPython:mainfrom
fanninpm:edit-dep-groups
Feb 28, 2026
Merged

Edit dependency groups#7257
youknowone merged 2 commits intoRustPython:mainfrom
fanninpm:edit-dep-groups

Conversation

@fanninpm
Copy link
Contributor

@fanninpm fanninpm commented Feb 28, 2026 •
edited by coderabbitai bot
Loading

So, as it turns out, in #7064 @dependabot didn't update all the random-related dependencies in a single PR.
Also, it might make more sense to fold the cranelift dependencies into the wasmtime group because cranelift is part of the wasmtime organization.

Summary by CodeRabbit

  • Chores
    • Updated internal dependency management configuration to reorganize how package updates are grouped and tracked. No impact to user-facing functionality.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 28, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 81acb9b and c80e067.

📒 Files selected for processing (1)
  • .github/dependabot.yml
📝 Walkthrough

Walkthrough

This pull request reorganizes Dependabot dependency groups in the configuration file. The cranelift group is removed and its crates (cranelift*) are consolidated into the wasmtime group. A new random group is introduced to manage random-number-generation related dependencies.

Changes

Cohort / File(s) Summary
Dependabot Configuration
.github/dependabot.yml		 Removed the cranelift group; migrated cranelift* crate patterns to the wasmtime group; created a new random group for getrandom, mt19937, and rand* crates.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested reviewers

  • youknowone

Poem

🐰 Hopping through dependencies with glee,
Cranelift finds its wasmtime tree,
Random numbers grouped up tight,
Dependabot keeps things right! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Edit dependency groups' accurately reflects the main changes: reorganizing dependency groups in dependabot.yml by creating a random group, folding cranelift into wasmtime, and adjusting patterns.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

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 and usage tips.

@fanninpm fanninpm requested a review from youknowone February 28, 2026 02:43
@youknowone youknowone merged commit f5a071d into RustPython:main Feb 28, 2026
13 checks passed
@fanninpm fanninpm deleted the edit-dep-groups branch February 28, 2026 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@youknowone youknowone youknowone approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@fanninpm @youknowone