astral-sh/ruff

Consider analyze-graph for packages and producing graph?

Open

#13,431 opened on 2024年9月20日

GitHub で見る
 (8 comments) (0 reactions) (0 assignees)Rust (47,527 stars) (2,088 forks)batch import
analyzehelp wanted

説明

My Graph-Imports package has been a useful tool in my development. Now that Ruff has ruff analyze graph, it could be extended to supplant my tool if there's a desire. Suggestions:

  • Add a --package flag to switch from producing a graph of module imports to package imports (take the union over every module within a package?)
  • Add a --output_image flag to produce an actual image of the graph rather than text?

I realize this may be out of scope, but I thought I'd suggest it since it might be easy to do. (The source for graph-imports is quite short: https://github.com/NeilGirdhar/graph_imports/blob/main/graph_imports/__init__.py)

コントリビューターガイド

技術スタック
rustpython
領域
clitoolingdeveloper experience
Issue 種別
feature
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
half day
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
active
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
mostly clear
前提条件
Rust programmingFamiliarity with ruff CLIBasic graph concepts
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
55
調査方針
First, read the existing comments on the issue (#13431) to understand maintainer interest. Then examine the current implementation of the `ruff analyze graph` subcommand in the codebase, likely in `crates/ruff cli/src/commands/analyze.rs`. Consider how to add a ` package` flag by aggregating module imports per package. For ` output image`, evaluate available Rust graph visualization libraries or consider generating Graphviz DOT output. Review the linked graph imports package for reference on package level graph generation.