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

Draw deprecated references #7

Merged
merged 6 commits into from
Dec 10, 2020
Merged

Draw deprecated references #7

merged 6 commits into from
Dec 10, 2020

Conversation

samuelgiles
Copy link
Owner

This PR implements an idea raised in #6 where it was suggested that drawing deprecated references would be useful in helping reveal issues in an application's architecture.

image

Red arrows now indicate a deprecated reference from one package to another, in the above example our images component is referencing a class in the Application.

Copy link
Collaborator

@michaelwebb76 michaelwebb76 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@samuelgiles samuelgiles merged commit 1a72433 into main Dec 10, 2020
@samuelgiles samuelgiles deleted the deprecated-dependencies branch December 10, 2020 10:39
euglena1215 added a commit to euglena1215/graphwerk that referenced this pull request Nov 20, 2023
A change was made at samuelgiles#7 to draw deprecated dependencies,
now named `package_todo.yml` instead of `deprecated_references.yml`.

I thought about renaming `DeprecatedReferencesLoader`, but if people are still using the old packwerk,
it would be a breaking change, so I added the `PackageTodoLoader` instead of renaming it.
euglena1215 added a commit to euglena1215/graphwerk that referenced this pull request Nov 21, 2023
A change was made at samuelgiles#7 to draw deprecated dependencies,
now named `package_todo.yml` instead of `deprecated_references.yml`.

I thought about renaming `DeprecatedReferencesLoader`, but if people are still using the old packwerk,
it would be a breaking change, so I added the `PackageTodoLoader` instead of renaming it.
samuelgiles pushed a commit that referenced this pull request Jan 29, 2024
A change was made at #7 to draw deprecated dependencies,
now named `package_todo.yml` instead of `deprecated_references.yml`.

I thought about renaming `DeprecatedReferencesLoader`, but if people are still using the old packwerk,
it would be a breaking change, so I added the `PackageTodoLoader` instead of renaming it.
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