Skip to content

[Bug fix] Fix rule B909's panic when checking large loop blocks - #11772

Merged
MichaReiser merged 1 commit into
astral-sh:mainfrom
Embers-of-the-Fire:main
Jun 6, 2024
Merged

MichaReiser merged 1 commit into
astral-sh:mainfrom
Embers-of-the-Fire:main

Conversation

@Embers-of-the-Fire

@Embers-of-the-Fire Embers-of-the-Fire commented Jun 6, 2024 •

Copy link
Copy Markdown
Contributor

Summary

This pr should close #11771 by changing LoopMutationsVisitor::<field>branch from u8 to u32.

Test Plan

Tested using the example programme mentioned in the related issue.

@codspeed

codspeed Bot commented Jun 6, 2024

Copy link
Copy Markdown

CodSpeed Performance Report

Merging #11772 will improve performances by 4.67%

Comparing Embers-of-the-Fire:main (06e69e0) with main (6c1fa1d)

Summary

⚡ 1 improvements
✅ 29 untouched benchmarks

Benchmarks breakdown

Benchmark main Embers-of-the-Fire:main Change
⚡ linter/all-with-preview-rules[numpy/globals.py] 839.1 µs 801.7 µs +4.67%

@github-actions

github-actions Bot commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@MichaReiser

Copy link
Copy Markdown
Member

Thanks

@MichaReiser
MichaReiser merged commit f144ede into astral-sh:main Jun 6, 2024
@dhruvmanila dhruvmanila added the bug An issue describing something that isn't working, or a PR that fixes a bug label Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug An issue describing something that isn't working, or a PR that fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rule B909 cause panic

3 participants