Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: ignore pruned nodes #103

Closed
wants to merge 1 commit into from
Closed

Conversation

gitphill
Copy link
Contributor

In ecosystems where the dep-graph contains pruned nodes, the dep-graph library functions like pkgPathsToRoot and countPathsToRoot can safely ignore those pruned nodes.

This is because there should be a path to a non-pruned node, the effective path.

Ignoring pruned paths has a number of advantages. Consuming applications are presented with less paths to traverse when operating over paths. When displaying dep-graph path information there is less noise as only the effective paths are shown.

In ecosystems where the dep-graph contains pruned nodes, the dep-graph
library functions like pkgPathsToRoot and countPathsToRoot can safely
ignore those pruned nodes.

This is because there should be a path to a non-pruned node, the
effective path.

Ignoring pruned paths has a number of advantages. Consuming applications
are presented with less paths to traverse when operating over paths.
When displaying dep-graph path information there is less noise as only
the effective paths are shown.
@gitphill gitphill closed this Mar 8, 2023
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.

None yet

1 participant