[ty] add cycle detection for find_legacy_typevars by carljm · Pull Request #20124 · astral-sh/ruff (original) (raw)

@carljm

@carljm added the ty

Multi-file analysis & type inference

label

Aug 28, 2025

@carljm

@carljm

@carljm

@carljm carljm marked this pull request as ready for review

August 28, 2025 03:05

AlexWaygood

dcreager

@carljm carljm deleted the cjm/findlegacycycle branch

August 28, 2025 16:55

dcreager added a commit that referenced this pull request

Aug 28, 2025

@dcreager

second-ed pushed a commit to second-ed/ruff that referenced this pull request

Sep 9, 2025

@carljm @second-ed

Summary

Add cycle detection to the find_legacy_typevars type method.

Test Plan

Added mdtest that stack overflowed without this.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})