Move populate cache option to cli group

core/dbt/cli/main.py

@@ -148,6 +148,7 @@ def invoke(self, args: List[str], **kwargs) -> dbtRunnerResult:
 @p.partial_parse
 @p.print
 @p.deprecated_print
+@p.populate_cache
 @p.printer_width
 @p.quiet
 @p.record_timing_info
@@ -178,7 +179,6 @@ def cli(ctx, **kwargs):
 @p.deprecated_favor_state
 @p.full_refresh
 @p.indirect_selection
-@p.populate_cache
 @p.profile
 @p.profiles_dir
 @p.project_dir
@@ -216,7 +216,6 @@ def build(ctx, **kwargs):
 # dbt clean
 @cli.command("clean")
 @click.pass_context
-@p.populate_cache
 @p.profile
 @p.profiles_dir
 @p.project_dir
@@ -251,7 +250,6 @@ def docs(ctx, **kwargs):
 @p.exclude
 @p.favor_state
 @p.deprecated_favor_state
-@p.populate_cache
 @p.profile
 @p.profiles_dir
 @p.project_dir
@@ -287,7 +285,6 @@ def docs_generate(ctx, **kwargs):
 @docs.command("serve")
 @click.pass_context
 @p.browser
-@p.populate_cache
 @p.port
 @p.profile
 @p.profiles_dir
@@ -326,7 +323,6 @@ def docs_serve(ctx, **kwargs):
 @p.indirect_selection
 @p.introspect
 @p.parse_only
-@p.populate_cache
 @p.profile
 @p.profiles_dir
 @p.project_dir
@@ -374,7 +370,6 @@ def compile(ctx, **kwargs):
 @p.indirect_selection
 @p.introspect
 @p.parse_only
-@p.populate_cache
 @p.profile
 @p.profiles_dir
 @p.project_dir
@@ -412,7 +407,6 @@ def show(ctx, **kwargs):
 @cli.command("debug")
 @click.pass_context
 @p.config_dir
-@p.populate_cache
 @p.profile
 @p.profiles_dir_exists_false
 @p.project_dir
@@ -436,7 +430,6 @@ def debug(ctx, **kwargs):
 # dbt deps
 @cli.command("deps")
 @click.pass_context
-@p.populate_cache
 @p.profile
 @p.profiles_dir
 @p.project_dir
@@ -459,7 +452,6 @@ def deps(ctx, **kwargs):
 @click.pass_context
 # for backwards compatibility, accept 'project_name' as an optional positional argument
 @click.argument("project_name", required=False)
-@p.populate_cache
 @p.profile
 @p.profiles_dir
 @p.project_dir
@@ -485,7 +477,6 @@ def init(ctx, **kwargs):
 @p.models
 @p.output
 @p.output_keys
-@p.populate_cache
 @p.profile
 @p.profiles_dir
 @p.project_dir
@@ -525,7 +516,6 @@ def list(ctx, **kwargs):
 @cli.command("parse")
 @click.pass_context
 @p.compile_parse
-@p.populate_cache
 @p.profile
 @p.profiles_dir
 @p.project_dir
@@ -557,7 +547,6 @@ def parse(ctx, **kwargs):
 @p.exclude
 @p.fail_fast
 @p.full_refresh
-@p.populate_cache
 @p.profile
 @p.profiles_dir
 @p.project_dir
@@ -594,7 +583,6 @@ def run(ctx, **kwargs):
 @click.pass_context
 @click.argument("macro")
 @p.args
-@p.populate_cache
 @p.profile
 @p.profiles_dir
 @p.project_dir
@@ -624,7 +612,6 @@ def run_operation(ctx, **kwargs):
 @click.pass_context
 @p.exclude
 @p.full_refresh
-@p.populate_cache
 @p.profile
 @p.profiles_dir
 @p.project_dir
@@ -664,7 +651,6 @@ def seed(ctx, **kwargs):
 @p.exclude
 @p.favor_state
 @p.deprecated_favor_state
-@p.populate_cache
 @p.profile
 @p.profiles_dir
 @p.project_dir
@@ -706,7 +692,6 @@ def source(ctx, **kwargs):
 @click.pass_context
 @p.exclude
 @p.output_path  # TODO: Is this ok to re-use?  We have three different output params, how much can we consolidate?
-@p.populate_cache
 @p.profile
 @p.profiles_dir
 @p.project_dir
@@ -752,7 +737,6 @@ def freshness(ctx, **kwargs):
 @p.favor_state
 @p.deprecated_favor_state
 @p.indirect_selection
-@p.populate_cache
 @p.profile
 @p.profiles_dir
 @p.project_dir