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

Move `derive` -> `crates/derive` by ShaharNaveh · Pull Request #6264 · RustPython/RustPython · GitHub

/ RustPython Public

Move derive -> crates/derive - #6264

Merged
youknowone merged 8 commits into
RustPython:mainfrom
ShaharNaveh:move-derive
Nov 15, 2025
ConversationCommits (8)ChecksFiles changed
Merged

Move derive -> crates/derive#6264
youknowone merged 8 commits into
RustPython:mainfrom
ShaharNaveh:move-derive

Conversation

ShaharNaveh commented Nov 14, 2025 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Contributor

ref: #6214

Summary by CodeRabbit

  • Chores
    • Reorganized internal package layout and adjusted internal dependency references to reflect the new layout.
    • Removed an internal workspace member that is no longer part of the build configuration.
    • No user-facing changes or functionality impacts.

coderabbitai Bot commented Nov 14, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Walkthrough

Workspace configuration updated: the derive crate was removed from the top-level workspace members and the rustpython-derive workspace dependency path changed from "derive" to "crates/derive", version unchanged.

Changes

Cohort / File(s) Change Summary
Workspace manifest
Cargo.toml		 Removed derive from [workspace] members; changed rustpython-derive dependency path from path = "derive" to path = "crates/derive" (version remains 0.4.0).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Small, localized manifest edits; verify workspace member removal and dependency path update.
  • Check CI/workspace build and any references to the old derive path in scripts or docs.

Possibly related PRs

Suggested reviewers

  • youknowone

Poem

🐰 Hop, hop — a crate on the move,
From root to crates it finds its groove,
Paths adjusted, tidy and neat,
Build tree humming with tiny feet. 🥕

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title directly and accurately describes the main change: moving the 'derive' crate from the root directory to 'crates/derive'.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches 🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📜 Recent review details

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a8954e4 and 1ce2f09.

📒 Files selected for processing (1)
  • Cargo.toml (1 hunks)
🔇 Additional comments (1)
Cargo.toml (1)

143-143: Derive crate relocation is complete and correctly configured.

All verification checks passed:

  • ✓ Crate successfully moved to crates/derive/ with valid metadata
  • ✓ Stale derive/ directory removed
  • ✓ No remaining references to old path = "derive" in manifests
  • ✓ Cargo.lock synchronized and consistent
  • ✓ Workspace dependency path correctly updated on line 143
  • ✓ Glob pattern crates/* properly captures relocated crate

The change integrates correctly with the broader workspace reorganization.

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.

ShaharNaveh mentioned this pull request Nov 14, 2025
Open
16 tasks

fanninpm commented Nov 14, 2025

Copy link
Copy Markdown
Contributor

@coderabbitai review

coderabbitai Bot commented Nov 14, 2025

Copy link
Copy Markdown
Contributor
✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

coderabbitai Bot reviewed Nov 14, 2025
View reviewed changes

coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Actionable comments posted: 1

📜 Review details

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 609d99f and dc7df08.

📒 Files selected for processing (2)
  • Cargo.toml (1 hunks)
  • jit/Cargo.toml (1 hunks)
🔇 Additional comments (1)
Cargo.toml (1)

154-154: Migration verified—no issues remain.

The crate relocation is complete and correct:

  • ✓ crates/derive/Cargo.toml exists with valid structure
  • ✓ No stale path = "derive" references found
  • ✓ All consumers (jit, pylib, stdlib, vm) use workspace = true
  • ✓ Root Cargo.toml line 154 correctly references path = "crates/derive"
  • ✓ Workspace glob pattern crates/* captures the moved crate
  • ✓ derive-impl (separate crate at root level) correctly remains an explicit member

Comment thread jit/Cargo.toml Outdated
youknowone merged commit 8715ae7 into RustPython:main Nov 15, 2025
13 checks passed
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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

coderabbitai[bot] coderabbitai[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL