Skip to content

Conversation

@bhearsum
Copy link
Contributor

Especially in repositories with large graphs, it can be useful to visualize the relationship between the various kinds.

I looked at directly outputting svg files of the graphs here, but decided against it because rendering mermaid diagrams requires a browser or puppeteer AFAICT; and I don't want to add that dependency.

I also looked at adding support for visualizing entire graphs, but those quickly get very unwieldy and I decided against it. (This might be useful in conjunction with a new --target-task option, but that's more than I'm willing to take on at this time.)

@bhearsum bhearsum marked this pull request as ready for review October 23, 2025 20:44
@bhearsum bhearsum requested a review from a team as a code owner October 23, 2025 20:44
@bhearsum bhearsum requested a review from ahal October 23, 2025 20:44
Copy link
Collaborator

@ahal ahal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks nifty!

"""
The dependency graph of kinds.
@type: Graph
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was going to comment that this isn't following the standard docstring format we do everywhere else (e.g), but then noticed we do use this above. We should make this consistent across the repo, but for now I agree it's better to be consistent within the file.

Especially in repositories with large graphs, it can be useful to visualize the relationship between the various kinds.

I looked at directly outputting svg files of the graphs here, but decided against it because rendering mermaid diagrams requires a browser or puppeteer AFAICT; and I don't want to add that dependency.

I also looked at adding support for visualizing entire graphs, but those quickly get very unwieldy and I decided against it. (This might be useful in conjunction with a new `--target-task` option, but that's more than I'm willing to take on at this time.)
@bhearsum bhearsum merged commit e0b8214 into taskcluster:main Oct 24, 2025
17 checks passed
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