Mercurial > kallithea
changeset 8816:c76638100ca0
deps: also report unseen known violations
| author | Mads Kiilerich <mads@kiilerich.com> |
|---|---|
| date | Tue, 29 Dec 2020 22:30:48 +0100 |
| parents | 1b683a4eb9fc |
| children | 233ba8198781 |
| files | scripts/deps.py |
| diffstat | 1 files changed, 4 insertions(+), 2 deletions(-) [+] |
line wrap: on
line diff
--- a/scripts/deps.py Mon Dec 28 00:43:18 2020 +0100 +++ b/scripts/deps.py Tue Dec 29 22:30:48 2020 +0100 @@ -153,7 +153,7 @@ shown_modules = normal_modules | top_modules # break the chains somehow - this is a cleanup TODO list -known_violations = [ +known_violations = set([ ('kallithea.lib.auth_modules', 'kallithea.lib.auth'), # needs base&facade ('kallithea.lib.utils', 'kallithea.model'), # clean up utils ('kallithea.lib.utils', 'kallithea.model.db'), @@ -166,7 +166,7 @@ ('kallithea.model', 'kallithea.lib.hooks'), # clean up hooks ('kallithea.model', 'kallithea.model.scm'), ('kallithea.model.scm', 'kallithea.lib.hooks'), -] +]) extra_edges = [ ('kallithea.config', 'kallithea.controllers'), # through TG @@ -271,6 +271,8 @@ # verify dependencies by untangling dependency chain bottom-up: todo = set(normalized_dep_edges) + unseen_violations = known_violations.difference(todo) + assert not unseen_violations, unseen_violations for x in known_violations: todo.remove(x)