Skip to content

Commit ddaa3e5

Browse files
authored
Fix typos and grammatical issues (#72)
* Fix typo: exludes -> excludes in test descriptions * Fix grammatical issues in documentation and tests
1 parent 2a6413d commit ddaa3e5

2 files changed

Lines changed: 7 additions & 7 deletions

File tree

lib/visualize_packs/options_parser.rb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,13 +26,13 @@ def self.parse(args)
2626
opt.on('--no-node-todos', "Don't show package-based todos") { |o| options.show_node_todos = false }
2727

2828
opt.on('--focus-pack=STRING', "Focus on a specific pack(s). Comma-separated list of packs. Wildcards supported: 'packs/*'") { |o| options.focus_pack = o.to_s.split(",") }
29-
opt.on('--focus-pack-edge-mode=STRING', "If focus-pack is set, this shows only between focussed packs (when set to none) or the edges into / out of / in and out of the focus packs to non-focus packs (which will be re-added to the graph). One of #{FocusPackEdgeDirection.values.map &:serialize}") { |o| options.show_only_edges_to_focus_pack = FocusPackEdgeDirection.deserialize(o) }
30-
opt.on('--exclude-packs=', "Exclude listed packs from diagram. If used with include you will get all included that are not excluded. Wildcards support: 'packs/ignores/*'") { |o| options.exclude_packs = o.to_s.split(",") }
29+
opt.on('--focus-pack-edge-mode=STRING', "If focus-pack is set, this shows only between focused packs (when set to none) or the edges into / out of / in and out of the focus packs to non-focus packs (which will be re-added to the graph). One of #{FocusPackEdgeDirection.values.map &:serialize}") { |o| options.show_only_edges_to_focus_pack = FocusPackEdgeDirection.deserialize(o) }
30+
opt.on('--exclude-packs=', "Exclude listed packs from diagram. If used with include you will get all included that are not excluded. Wildcards supported: 'packs/ignores/*'") { |o| options.exclude_packs = o.to_s.split(",") }
3131

3232
opt.on('--roll-nested-into-parent-packs', "Don't show nested packs (not counting root). Connect edges to top-level pack instead") { |o| options.roll_nested_into_parent_packs = true }
3333
opt.on('--no-nesting-arrows', "Don't draw relationships between parents and nested packs") { |o| options.show_nested_relationships = false }
3434

35-
opt.on('--remote-base-url=STRING', "Link pack packs to a URL (affects graphviz SVG generation)") { |o| options.remote_base_url = o }
35+
opt.on('--remote-base-url=STRING', "Link packs to a URL (affects graphviz SVG generation)") { |o| options.remote_base_url = o }
3636

3737
opt.on('--title=STRING', "Set a custom diagram title") { |o| options.title = o }
3838

spec/visualize_packs_spec.rb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -429,10 +429,10 @@
429429
#Filtering + excluding (including wildcards)
430430
['a1234c', true_, true_, 'dpav', 'a', ' ', ' ', nil, %w(packs/a/1), 'a 234c', 'exclude_packs removes node'],
431431
['a1234c', true_, true_, 'dpav', 'a', ' ', ' ', nil, %w(packs/a/*), 'a 34c', 'exclude_packs with wildcard removes nodes'],
432-
['a1234c', true_, true_, 'dpav', 'a', ' ', ' ', nil, %w(packs/a packs/a/*), ' 34c', 'exclude_packs with multiple exludes works'],
433-
['a1234c', true_, true_, 'dpav', 'a', ' ', ' ', %w(packs/a), %w(packs/a), ' ', 'exclude_packs exludes focus'],
434-
['a1234c', true_, true_, 'dpav', 'a', 'a1 ', ' ', %w(packs/a), %w(packs/a), ' 1 ', 'exclude_packs exludes focus but keeps dependency'],
435-
['a1234c', true_, true_, 'dpav', 'a', ' ', ' ad1', %w(packs/a), %w(packs/a), ' 1 ', 'exclude_packs exludes focus but keeps todo'],
432+
['a1234c', true_, true_, 'dpav', 'a', ' ', ' ', nil, %w(packs/a packs/a/*), ' 34c', 'exclude_packs with multiple excludes works'],
433+
['a1234c', true_, true_, 'dpav', 'a', ' ', ' ', %w(packs/a), %w(packs/a), ' ', 'exclude_packs excludes focus'],
434+
['a1234c', true_, true_, 'dpav', 'a', 'a1 ', ' ', %w(packs/a), %w(packs/a), ' 1 ', 'exclude_packs excludes focus but keeps dependency'],
435+
['a1234c', true_, true_, 'dpav', 'a', ' ', ' ad1', %w(packs/a), %w(packs/a), ' 1 ', 'exclude_packs excludes focus but keeps todo'],
436436
#Filtering + dependencies + todos + excluding
437437
['a1234c', true_, true_, ' pa ', 'o', 'a1 2a', ' ad3 4aa apc', %w(packs/a), %w(packs/a), ' 1 c', 'combination of todo filtering, edge mode, focus, and exclude works'],
438438
].each do |c|

0 commit comments

Comments
 (0)