seddonym / grimp

Builds a graph of a Python project's internal dependencies.
https://grimp.readthedocs.io
BSD 2-Clause "Simplified" License
51 stars 10 forks source link

Correct error with downstream/upstream #121

Closed seddonym closed 11 months ago

seddonym commented 11 months ago

This PR corrects a mistake in which find_illegal_dependencies_for_layers, and the PackageDependency object, reversed the meaning of downstream/upstream in a way that was inconsistent with the rest of Grimp.

It's likely that no-one is using this method yet so I'm planning to be pragmatic and just fix it in a new release.