Update ruff to `0.15.6` by ShaharNaveh · Pull Request #7427 · RustPython/RustPython

ShaharNaveh

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

The PR updates the Ruff toolchain dependency from tag 0.15.5 to 0.15.6 with corresponding git revision updates and adds placeholder support for lazy imports by introducing new is_lazy fields in tokenization and AST statement handling.

Changes

Cohort / File(s)	Summary
Dependency Update `Cargo.toml`	Bumped Ruff toolchain from 0.15.5 to 0.15.6; updated four Ruff-related git dependencies (ruff_python_parser, ruff_python_ast, ruff_text_size, ruff_source_file) to commit e4c7f357777a2fdd34dbe6a98b1b7d3e7488f675.
Lazy Import Support `crates/stdlib/src/_tokenize.rs`, `crates/vm/src/stdlib/_ast/statement.rs`	Added placeholder support for lazy imports: new match arm for TokenKind::Lazy returning u8::MAX; added is_lazy field (bool) to StmtImport and StmtImportFrom AST nodes, both initialized to false.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Update ruff to 0.15.0 #7086: Updates the same Ruff git dependencies in Cargo.toml to a newer revision.
Update ruff to 0.15.1 #7105: Modifies Ruff-related git dependency entries in Cargo.toml with version bumps.
Update to ruff 0.14.14 #6861: Bumps Ruff toolchain and associated crate dependencies in Cargo.toml.

Suggested reviewers

youknowone

Poem

🐰 Lazy hops are on the way,
New fields blooming, fresh today,
Ruff now points to newer ground,
Placeholders hold what's soon to be found,
Version bumps and imports set,
The best improvements yet! 🌟

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The pull request title 'Update ruff to `0.15.6`' directly aligns with the primary change in Cargo.toml, which bumps the Ruff toolchain from 0.15.5 to 0.15.6. However, the changeset also includes unrelated modifications to token handling and AST structures that are not reflected in the title.

_{✏️ 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

📝 Coding Plan

Generate coding plan for human review comments

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