Skip to content

Move wtf8 -> crates/wtf8 - #6257

Merged
youknowone merged 1 commit into
RustPython:mainfrom
ShaharNaveh:move-wtf8
Nov 14, 2025
Merged

Move wtf8 -> crates/wtf8#6257
youknowone merged 1 commit into
RustPython:mainfrom
ShaharNaveh:move-wtf8

Conversation

@ShaharNaveh

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

Copy link
Copy Markdown
Contributor

ref #6214

Summary by CodeRabbit

  • Chores
    • Internal project structure reorganizationβ€”no user-facing changes.

@coderabbitai

coderabbitai Bot commented Nov 14, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Walkthrough

The workspace configuration is updated to reorganize the wtf8 crate into a subdirectory structure. The "wtf8" member is removed from the workspace members list, and the rustpython-wtf8 dependency path is updated from "wtf8" to "crates/wtf8" while maintaining the existing version constraint.

Changes

Cohort / File(s) Change Summary
Workspace Configuration
Cargo.toml		 Removed "wtf8" from [workspace] members; updated rustpython-wtf8 dependency path from "wtf8" to "crates/wtf8"

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

  • Verify the wtf8 crate physically exists at crates/wtf8 location
  • Confirm no other Cargo.toml files or workspace references need updating

Poem

🐰 Hops along with glee

Into crates/ we organize,
Workspace tidy, nested and wise,
wtf8 finds its rightful place,
Dependencies dance with newfound grace!

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 describes the main change: moving the wtf8 crate from the root directory to the crates subdirectory, which aligns with the Cargo.toml modifications.
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 609d99f and 94c9084.

πŸ“’ Files selected for processing (1)
  • Cargo.toml (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
πŸ“š Learning: 2025-09-28T22:22:55.921Z 
Learnt from: reactive-firewall
Repo: RustPython/RustPython PR: 6176
File: .github/workflows/Check_Tests.yml:133-141
Timestamp: 2025-09-28T22:22:55.921Z
Learning: In the RustPython project's CI-5974-Test-RustPython-Integration action, the override-rustpython-path input is marked as required but has runtime fallback logic that defaults to RUSTPYTHONPATH environment variable or "Lib" if neither is provided, making explicit specification unnecessary in most cases.

Applied to files:

  • Cargo.toml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (11)
  • GitHub Check: Run snippets and cpython tests on wasm-wasi
  • GitHub Check: Check the WASM package and demo
  • GitHub Check: Run tests under miri
  • GitHub Check: Run snippets and cpython tests (windows-latest)
  • GitHub Check: Run snippets and cpython tests (ubuntu-latest)
  • GitHub Check: Run snippets and cpython tests (macos-latest)
  • GitHub Check: Run rust tests (ubuntu-latest)
  • GitHub Check: Ensure compilation on various targets
  • GitHub Check: Run rust tests (windows-latest)
  • GitHub Check: Run rust tests (macos-latest)
  • GitHub Check: auto_format
πŸ”‡ Additional comments (1)
Cargo.toml (1)

162-162: Dependency path correctly updated to reflect new crate location.

Migration verified complete and correct. The rustpython-wtf8 path on line 162 has been properly updated to "crates/wtf8", the workspace members list includes the "crates/*" glob pattern to auto-discover the relocated crate, and no stale references to the old path remain. The crate structure is now consistent with other crates under crates/.

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 ShaharNaveh mentioned this pull request Nov 14, 2025
Open
16 tasks
@youknowone
youknowone merged commit 3728879 into RustPython:main Nov 14, 2025
13 checks passed
This was referenced Nov 15, 2025
Merged
Merged
Merged
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

@ShaharNaveh @youknowone