◐ Shell
clean mode source ↗

Move `compiler/source` -> `crates/compiler-source` by ShaharNaveh · Pull Request #6261 · RustPython/RustPython

ShaharNaveh

Walkthrough

CI workflow configuration updated to exclude two additional crates (rustpython_wasm and rustpython-compiler-source) from clippy checks and rust test execution during continuous integration runs.

Changes

Cohort / File(s) Summary
CI workflow configuration
.github/workflows/ci.yaml		 Added rustpython_wasm and rustpython-compiler-source to clippy exclusion list and rust test command exclusions

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Straightforward configuration additions with repetitive pattern (same crates added to multiple exclusion lists)

Suggested reviewers

  • youknowone

Poem

🐰 A rabbit hops through CI workflows today,
Two crates now skip the checker's way,
Wasm and compiler sources rest,
While clippy continues its quest! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The PR title describes moving compiler/source to crates/compiler-source, but the changes only update CI workflow exclusions for rustpython_wasm and rustpython-compiler-source without reflecting the actual directory move mentioned in the title. Clarify whether the title accurately represents all changes made, or confirm if directory restructuring commits are pending or in different branches.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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 42624a5 and 9c0ccfb.

📒 Files selected for processing (1)
  • .github/workflows/ci.yaml (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • .github/workflows/ci.yaml
⏰ 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). (12)
  • GitHub Check: Check the WASM package and demo
  • GitHub Check: Check Rust code with clippy
  • 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 rust tests (ubuntu-latest)
  • GitHub Check: Run snippets and cpython tests (macos-latest)
  • GitHub Check: Run rust tests (windows-latest)
  • GitHub Check: Run snippets and cpython tests on wasm-wasi
  • GitHub Check: Run rust tests (macos-latest)
  • GitHub Check: Ensure compilation on various targets
  • GitHub Check: auto_format

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.