[nekomario28]

Summary

Fixes #29070.

TID254 incorrectly flags eager imports in stub files when require-lazy is configured. Skip the rule for stubs.

Test Plan

Added a pyi block covering require-lazy and ban-lazy for both import and from ... import. The lazy-import-mismatch mdtest passes.

[ntBre]

Thanks, this looks right to me. I just had a suggestion about the tests.

[ntBre]

Let's use mdtests for this:

ruff/CONTRIBUTING.md

Lines 233 to 240 in bd2d6f6

	#### Rule testing: fixtures and snapshots
	To test rules, Ruff uses the mdtest framework, initially developed for ty. Mdtests are written as
	Markdown files with Python code and TOML configuration blocks alongside prose
	descriptions. Generally, there will be one directory per linter (e.g. `flake8-bandit` for the `S`
	rules) with nested files for each rule (e.g. `unsafe-markup-use.md` for `S704`). Within these
	files, you can define additional Markdown sections to group related tests and their settings
	together.

You should be able to use a pyi block in the existing mdtest file and avoid needing this setup function.

[nekomario28]

Thanks! Moved the stub tests into a pyi block in the existing mdtest.

[astral-sh-bot]

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

[ntBre]

Thanks!