feat!(extract): makes 'real' depencency type available alongside aliases BREAKING #856
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description, Motivation and Context
Before this PR dependency types that came out of an alias (subpath import, webpack alias, tsconfig path) only stated that the dependency was an alias. However, these aliases point to other dependency types - local modules, third party ('npm') modules, nodejs builtin core(!) modules. This is useful information, and actually not listing those is an omission.
After merging this PR dependency-cruiser will start listing the 'actual' dependency type alongside the alias dependency types.
Technically this is a BREAKING CHANGE because rules that triggered on these 'real' dependency types before didn't trigger because they weren't detected.
after
before
How Has This Been Tested?
Types of changes
Checklist
📖
⚖️