Merge pull request #5957 from fmssn/enhancement-5939-fix-noisy-output…

…-pipenv-check

set regular logging level to ERROR

README.md

@@ -337,4 +337,3 @@ Documentation resides over at [pipenv.pypa.io](https://pipenv.pypa.io/en/latest/
     <img alt="Star History Chart" src="https://api.star-history.com/svg?repos=pypa/pipenv&type=Date" />
   </picture>
 </a>
-

pipenv/cli/command.py

@@ -520,6 +520,7 @@ def check(
         output=output,
         key=key,
         quiet=quiet,
+        verbose=state.verbose,
         exit_code=exit_code,
         policy_file=policy_file,
         save_json=save_json,

pipenv/routines/check.py

@@ -1,5 +1,6 @@
 import io
 import json as simplejson
+import logging
 import os
 import sys
 import tempfile
@@ -12,6 +13,40 @@
 from pipenv.vendor import click, plette
 
 
+def build_options(
+    audit_and_monitor=True,
+    exit_code=True,
+    output="screen",
+    save_json="",
+    policy_file="",
+    safety_project=None,
+    temp_requirements_name="",
+):
+    options = [
+        "--audit-and-monitor" if audit_and_monitor else "--disable-audit-and-monitor",
+        "--exit-code" if exit_code else "--continue-on-error",
+    ]
+    formats = {"full-report": "--full-report", "minimal": "--json"}
+
+    if output in formats:
+        options.append(formats.get(output, ""))
+    elif output not in ["screen", "default"]:
+        options.append(f"--output={output}")
+
+    if save_json:
+        options.append(f"--save-json={save_json}")
+
+    if policy_file:
+        options.append(f"--policy-file={policy_file}")
+
+    if safety_project:
+        options.append(f"--project={safety_project}")
+
+    options.extend(["--file", temp_requirements_name])
+
+    return options
+
+
 def do_check(
     project,
     python=False,
@@ -21,6 +56,7 @@ def do_check(
     output="screen",
     key=None,
     quiet=False,
+    verbose=False,
     exit_code=True,
     policy_file="",
     save_json="",
@@ -32,6 +68,9 @@ def do_check(
 ):
     import json
 
+    if not verbose:
+        logging.getLogger("pipenv").setLevel(logging.WARN)
+
     if not system:
         # Ensure that virtualenv is available.
         ensure_project(
@@ -120,27 +159,6 @@ def do_check(
     else:
         ignored = []
 
-    options = [
-        "--audit-and-monitor" if audit_and_monitor else "--disable-audit-and-monitor",
-        "--exit-code" if exit_code else "--continue-on-error",
-    ]
-
-    formats = {"full-report": "--full-report", "minimal": "--json"}
-    if output in formats:
-        options.append(formats.get(output, ""))
-
-    elif output not in ["screen", "default"]:
-        options.append(f"--output={output}")
-
-    if save_json:
-        options.append(f"--save-json={save_json}")
-
-    if policy_file:
-        options.append(f"--policy-file={policy_file}")
-
-    if safety_project:
-        options.append(f"--project={safety_project}")
-
     if use_installed:
         target_venv_packages = run_command(
             _cmd + ["-m", "pip", "list", "--format=freeze"],
@@ -165,7 +183,15 @@ def do_check(
     temp_requirements.write(target_venv_packages.stdout.strip())
     temp_requirements.close()
 
-    options.extend(["--file", temp_requirements.name])
+    options = build_options(
+        audit_and_monitor=audit_and_monitor,
+        exit_code=exit_code,
+        output=output,
+        save_json=save_json,
+        policy_file=policy_file,
+        safety_project=safety_project,
+        temp_requirements_name=temp_requirements.name,
+    )
 
     cmd = _cmd + [safety_path, "check"] + options