Skip to content

Commit

Permalink
Add logging for skipped resources (#3833)
Browse files Browse the repository at this point in the history
Add --greedy flag to subparser

Add greedy flag, override default behaviour when parsing union

Add greedy support to ls (I think?!)

That was suspiciously easy

Fix flake issues

Try adding tests with greedy support

Remove trailing whitespace

Fix return type for select_nodes

forgot to add greedy arg

Remove incorrectly expected test

Use named param for greedy

Pull alert_unused_nodes out into its own function

rename resources -> tests

Add expand_selection explanation of --greedy flag

Add greedy support to yaml selectors. Update warning, tests

Fix failing tests

Fix tests. Changelog

Fix flake8

Co-authored-by: Joel Labes c/o Jeremy Cohen <jeremy@dbtlabs.com>

automatic commit by git-black, original commits:
  6563d09
  • Loading branch information
joellabes authored and iknox-fa committed Feb 8, 2022
1 parent 2d0582e commit f81a660
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion core/dbt/graph/selector.py
Original file line number Diff line number Diff line change
Expand Up @@ -254,7 +254,7 @@ def get_selected(self, spec: SelectionSpec) -> Set[UniqueId]:
- node selection. Based on the include/exclude sets, the set
of matched unique IDs is returned
- includes direct + indirect selection (for tests)
- includes direct + indirect selection (for tests)
- filtering:
- selectors can filter the nodes after all of them have been
selected
Expand Down
6 changes: 3 additions & 3 deletions core/dbt/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -394,7 +394,7 @@ def _build_build_subparser(subparsers, base_subparser):
choices=['eager', 'cautious'],
default="eager",
dest='indirect_selection',
help='''
help="""
Select all tests that are adjacent to selected resources,
even if they those resources have been explicitly selected.
''',
Expand Down Expand Up @@ -741,7 +741,7 @@ def _build_test_subparser(subparsers, base_subparser):
choices=['eager', 'cautious'],
default="eager",
dest='indirect_selection',
help='''
help="""
Select all tests that are adjacent to selected resources,
even if they those resources have been explicitly selected.
''',
Expand Down Expand Up @@ -846,7 +846,7 @@ def _build_list_subparser(subparsers, base_subparser):
choices=['eager', 'cautious'],
default="eager",
dest='indirect_selection',
help='''
help="""
Select all tests that are adjacent to selected resources,
even if they those resources have been explicitly selected.
''',
Expand Down

0 comments on commit f81a660

Please sign in to comment.