kallithea Changeset - c76638100ca0

Changeset - c76638100ca0

Parent rev.

Child rev.

[Not reviewed]

default

0 1 0

Mads Kiilerich (mads) - 5 years ago 2020-12-29 22:30:48
mads@kiilerich.com

deps: also report unseen known violations

1 file changed with 4 insertions and 2 deletions:

scripts/deps.py

0 comments (0 inline, 0 general)

scripts/deps.py

➞

Show inline comments

@@ @@ -153,7 +153,7 @@ kallithea.templates.py @@
 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 @@ known_violations = [ @@
 ('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 @@ Usage: @@
     # 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)

0 comments (0 inline, 0 general)