Add —use-todos-for-layout to configure this explicitly (#51)

Author: shageman

bin/visualize_packs

@@ -19,6 +19,7 @@ OptionParser.new do |opt|
 
   opt.on('--no-todo-arrows', "Don't show pack todos") { |o| options.show_todos = false }
   opt.on("--only-todo-types=", "Show only the selected types of todos. Comma-separated list of #{EdgeTodoTypes.values.map &:serialize}") { |o| options.only_todo_types = o.to_s.split(",").uniq.map { EdgeTodoTypes.deserialize(_1) } }
+  opt.on("--use-todos-for-layout", "Show only the selected types of todos. Comma-separated list of #{EdgeTodoTypes.values.map &:serialize}") { |o| options.use_todos_for_layout = true }
 
   opt.on('--no-teams', "Don't show team colors") { |o| options.show_teams = false }
 

lib/graph.dot.erb

@@ -88,7 +88,7 @@ digraph package_diagram {
         <%- todo_types.keys.each do |todo_type| -%>
           <%- if show_edge.call(package.name, todos_to_package) -%>
             "<%= package.name -%>" -> "<%= todos_to_package -%>"<%= todo_type == 'privacy' ? ':private' : '' -%> [ color=darkred style=dashed
-              <%- if options.show_dependencies -%>
+              <%- if !options.use_todos_for_layout -%>
                 constraint=false
               <%- end -%>
               # headlabel="<%= todo_type -%>"

lib/visualize_packs/options.rb

@@ -32,6 +32,7 @@ class Options < T::Struct
 
   prop :show_todos, T::Boolean, default: true
   prop :only_todo_types, T::Array[EdgeTodoTypes], default: EdgeTodoTypes.values
+  prop :use_todos_for_layout, T::Boolean, default: false
 
   prop :show_teams, T::Boolean, default: true
 

spec/diagram_generation_helper.sh

@@ -19,7 +19,7 @@ test_names+=("no_legend"); test_params[${test_names[-1]}]="--no-legend"
 test_names+=("no_layers"); test_params[${test_names[-1]}]="--no-layers"
 test_names+=("no_teams"); test_params[${test_names[-1]}]="--no-teams"
 
-test_names+=("no_dependencies"); test_params[${test_names[-1]}]="--no-dependency-arrows"
+test_names+=("no_dependencies"); test_params[${test_names[-1]}]="--no-dependency-arrows --use-todos-for-layout"
 test_names+=("no_todos"); test_params[${test_names[-1]}]="--no-todo-arrows"
 test_names+=("no_privacy"); test_params[${test_names[-1]}]="--no-privacy-boxes"
 test_names+=("no_nested"); test_params[${test_names[-1]}]="--no-nesting-arrows"