Skip to content

Extract import analysis into ruff_python_imports#24757

Draft
zsol wants to merge 1 commit intomainfrom
zsol/jj-prqzlpouzmlx
Draft

Extract import analysis into ruff_python_imports#24757
zsol wants to merge 1 commit intomainfrom
zsol/jj-prqzlpouzmlx

Conversation

@zsol
Copy link
Copy Markdown
Member

@zsol zsol commented Apr 20, 2026

This provides a clean API for import analysis that can be used by both Ruff and uv, by moving the import collector, resolver, database setup, and string-import settings out of ruff_graph into a dedicated crate.

@astral-sh-bot
Copy link
Copy Markdown

astral-sh-bot Bot commented Apr 20, 2026

Typing conformance results

No changes detected ✅

Current numbers
The percentage of diagnostics emitted that were expected errors held steady at 87.94%. The percentage of expected errors that received a diagnostic held steady at 83.36%. The number of fully passing files held steady at 79/133.

@astral-sh-bot
Copy link
Copy Markdown

astral-sh-bot Bot commented Apr 20, 2026

Memory usage report

Memory usage unchanged ✅

@astral-sh-bot
Copy link
Copy Markdown

astral-sh-bot Bot commented Apr 20, 2026

ecosystem-analyzer results

No diagnostic changes detected ✅

Full report with detailed diff (timing results)

@MichaReiser
Copy link
Copy Markdown
Member

Just a heads up, ruff analyze isn't very accurate. It doesn't correctly handle submodule access because it doesn't do full type analysis.

This provides a clean API for import analysis that can be used by both Ruff and uv, by
moving the import collector, resolver, database setup, and string-import
settings out of ruff_graph into a dedicated crate.
@zsol zsol force-pushed the zsol/jj-prqzlpouzmlx branch from e51e7b9 to 07962c0 Compare April 20, 2026 20:55
@astral-sh-bot
Copy link
Copy Markdown

astral-sh-bot Bot commented Apr 20, 2026

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Formatter (stable)

✅ ecosystem check detected no format changes.

Formatter (preview)

✅ ecosystem check detected no format changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants