chore(eng-7893): removing duplicate warnings

Author: kieranmanning

cloudsmith_cli/cli/commands/repos.py

@@ -60,7 +60,6 @@ def print_repositories(opts, data, page_info=None, show_list_info=True):
 @decorators.common_cli_output_options
 @decorators.common_api_auth_options
 @decorators.initialise_api
-@decorators.verify_authenticated
 @click.pass_context
 def repositories(ctx, opts):  # pylink: disable=unused-argument
     """
@@ -76,6 +75,7 @@ def repositories(ctx, opts):  # pylink: disable=unused-argument
 @decorators.common_cli_output_options
 @decorators.common_api_auth_options
 @decorators.initialise_api
+@decorators.verify_authenticated
 @click.argument(
     "owner_repo",
     metavar="OWNER/REPO",

cloudsmith_cli/cli/config.py

@@ -179,10 +179,11 @@ def load_config(cls, opts, path=None, profile=None):
                 values = config["profile:%s" % profile]
                 cls._load_values_into_opts(opts, values)
             except KeyError:
-                click.secho(
-                    f"Warning: profile {profile} not found in config files {cls.config_files}",
-                    fg="yellow",
-                )
+                if not cls.config_already_warned():
+                    click.secho(
+                        f"Warning: profile {profile} not found in config files {cls.config_files}",
+                        fg="yellow",
+                    )
 
         existing_config_paths = {
             path: os.path.exists(path) for path in cls.config_files

cloudsmith_cli/cli/decorators.py

@@ -315,22 +315,35 @@ def call_print_rate_limit_info_with_opts(rate_info):
 def verify_authenticated(f):
     """Verify that the user is authenticated and warn if not."""
 
+    @click.option(
+        "--no-warn",
+        default=False,
+        is_flag=True,
+        help="Don't warn on misconfiguration issues",
+    )
     @click.pass_context
     @functools.wraps(f)
     def wrapper(ctx, *args, **kwargs):
         cloudsmith_host = kwargs["opts"].opts["api_config"].host
+        print(kwargs["opts"].opts["api_config"].__dict__)
+        no_warn = kwargs.pop("no_warn")
         is_auth, _, _, _ = get_user_brief()
-        if not is_auth:
+        if not is_auth and not no_warn:
             click.secho(
                 "Warning: You are not authenticated with the API. "
                 "Please verify your config files, API key and "
                 "run `cloudsmith login` if necessary to authenticate.",
                 fg="yellow",
             )
             click.secho(
-                f"You're currently attemping to connect to Cloudsmith instance {cloudsmith_host}",
+                f"You're currently attempting to connect to Cloudsmith instance {cloudsmith_host}",
                 fg="yellow",
             )
+            no_warn = True
+
+        opts = config.get_or_create_options(ctx)
+        opts.no_warn = no_warn
+        kwargs["opts"] = opts
         return ctx.invoke(f, *args, **kwargs)
 
     return wrapper