Move `compiler/codegen` -> `crates/codegen` by ShaharNaveh · Pull Request #6260 · RustPython/RustPython

ShaharNaveh

Walkthrough

The workspace manifest (Cargo.toml) was updated: the codegen workspace member was moved from compiler/codegen to crates/codegen, and the rustpython-codegen workspace dependency path was adjusted; the package version remains 0.4.0.

Changes

Cohort / File(s)	Change Summary
Workspace manifest `Cargo.toml`	Removed `compiler/codegen` from `[workspace]` members and changed `rustpython-codegen` workspace dependency path from `compiler/codegen` → `crates/codegen` (version unchanged: 0.4.0).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Single manifest edit; low logic complexity but affects workspace layout.
Review attention:
- Ensure no other workspace members or CI/scripting reference compiler/codegen.
- Confirm crates/codegen/Cargo.toml exists and has correct package metadata.

Possibly related PRs

Move common -> crates/common #6256 — Makes the same pattern of moving a package into crates/ and updating Cargo.toml paths.

Poem

🐰 A hop from compiler to crates I go,

Tiny paws patter, tidy rows in tow.
Codegen shifted, paths made right,
I nibble carrots, then rest tonight. 🥕✨

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 title directly and specifically describes the main change: moving the compiler/codegen directory from compiler/ to crates/ location, which matches the actual modifications in Cargo.toml.
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

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.}